web/notify.moe
Code Issues Pull Requests Activity

Removed incorrect comments

Browse Source
This commit is contained in:
Eduard Urbach 2017-06-23 19:26:54 +02:00
parent b5d2cc0465
commit 3286b3b7db
1 changed files with 0 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
middleware/UserInfo.go
View File

@ -37,8 +37,6 @@ func UserInfo() aero.Middleware {
} }
// Ignore API requests // Ignore API requests
// Note that API requests can filter data (privacy) and we might accidentally save the filtered data.
// That's why it's very important to ignore all API requests and not call user.Save() in this context.
if strings.HasPrefix(ctx.URI(), "/api/") { if strings.HasPrefix(ctx.URI(), "/api/") {
return return
} }
@ -50,11 +48,6 @@ func UserInfo() aero.Middleware {
return return
} }
// Let's be 100% sure we really do not accidentally save filtered data.
if user.Email == "" && user.IP == "" && user.FirstName == "" {
return
}
// This works asynchronously so it doesn't block the response // This works asynchronously so it doesn't block the response
go updateUserInfo(ctx, user) go updateUserInfo(ctx, user)
} }