mir/vinyldns
2
0
Fork 0
mirror of https://github.com/VinylDNS/vinyldns synced 2025-09-01 23:05:15 +00:00
Code Issues Releases Wiki Activity

add more logging around oidc login (#502)

Browse Source 
* add more logging around oidc login
This commit is contained in:
Rebecca Star
2019-02-21 11:45:51 -05:00
committed by GitHub
parent 00d767d5bf
commit d13b53a6be
5 changed files with 48 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
modules/portal/app/controllers/FrontendController.scala
View File

@@ -52,7 +52,7 @@ class FrontendController @Inject()(
case Some(_) => Redirect("/index")
case None =>
logger.info(s"No ${VinylDNS.ID_TOKEN} in session; Initializing oidc login")
Redirect(oidcAuthenticator.getCodeCall.toString)
Redirect(oidcAuthenticator.getCodeCall.toString, 302)
}
} else {
request.session.get("username") match {
@@ -70,9 +70,20 @@ class FrontendController @Inject()(
}
}
def logout(): Action[AnyContent] = Action { implicit request =>
private def getLoggedInUser(request: RequestHeader) =
if (oidcEnabled) {
request.session
.get(VinylDNS.ID_TOKEN)
.flatMap {
oidcAuthenticator.getValidUsernameFromToken
}
} else {
request.session.get("username")
}.getOrElse("No user in session")
def logout(): Action[AnyContent] = Action { implicit request =>
logger.info(s"Initializing logout for user [${getLoggedInUser(request)}]")
if (oidcEnabled) {
logger.info(s"Initializing oidc logout")
Redirect(oidcAuthenticator.oidcLogoutUrl).withNewSession
} else {
Redirect("/login").withNewSession
@@ -80,8 +91,7 @@ class FrontendController @Inject()(
}
def noAccess(): Action[AnyContent] = Action { implicit request =>
logger.info(
s"User account for '${request.session.get("username").getOrElse("username not found")}' is locked.")
logger.info(s"User account for '${getLoggedInUser(request)}' is locked.")
Unauthorized(views.html.noAccess())
}