diff options
author | Rémy Coutable <remy@rymai.me> | 2016-12-01 13:07:55 +0300 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2016-12-01 13:07:55 +0300 |
commit | c2be86b50f50d7bd5ec36b2893aa20d85dc3d21c (patch) | |
tree | c61e54a1a7000086116b38a2ec18f9346748ee12 /GITLAB_WORKHORSE_VERSION | |
parent | bbcce5f7dcbc498bb82851f07be6d9a41578ea28 (diff) | |
parent | 374033fe26013c685157ac0a3cd2a2b40f992ef5 (diff) |
Merge branch 'improve_oauth_user_error' into 'master'
Improve the `Gitlab::OAuth::User` error message
## What does this MR do?
Fixes a logging issue submitted by a customer at https://gitlab.zendesk.com/agent/tickets/50728.
The error saving the user is logged to application.log. Previously,
the entry had no context and was unusable - `Error saving user:
[Email address already taken]`. Adding the auth hash UID and email
makes the error more helpful.
For such a small logging change, do we need a changelog entry?
See merge request !7851
Diffstat (limited to 'GITLAB_WORKHORSE_VERSION')
0 files changed, 0 insertions, 0 deletions