Skip to content

Commit

Permalink
#264 Lower case user login on oauth
Browse files Browse the repository at this point in the history
  • Loading branch information
nadouani committed Oct 27, 2020
1 parent 1cf82e6 commit 619b28a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/org/thp/cortex/services/mappers/GroupUserMapper.scala
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ class GroupUserMapper(
.flatMap(o => (jsValue \ o).asOpt[String])
.orElse(defaultOrganization)
.fold[JsResult[String]](JsError())(o => JsSuccess(o))
} yield Fields(Json.obj("login" -> login, "name" -> name, "roles" -> roles, "organization" -> organization))
} yield Fields(Json.obj("login" -> login.toLowerCase, "name" -> name, "roles" -> roles, "organization" -> organization))
fields match {
case JsSuccess(f, _) => Future.successful(f)
case JsError(errors) => Future.failed(AuthenticationError(s"User info fails: ${errors.map(_._1).mkString}"))
Expand Down
2 changes: 1 addition & 1 deletion app/org/thp/cortex/services/mappers/SimpleUserMapper.scala
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ class SimpleUserMapper(
.flatMap(o => (jsValue \ o).asOpt[String])
.orElse(defaultOrganization)
.fold[JsResult[String]](JsError())(o => JsSuccess(o))
} yield Fields(Json.obj("login" -> login, "name" -> name, "roles" -> roles, "organization" -> organization))
} yield Fields(Json.obj("login" -> login.toLowerCase, "name" -> name, "roles" -> roles, "organization" -> organization))
fields match {
case JsSuccess(f, _) => Future.successful(f)
case JsError(errors) => Future.failed(AuthenticationError(s"User info fails: ${errors.map(_._1).mkString}"))
Expand Down

0 comments on commit 619b28a

Please sign in to comment.