Issue #7545 - changes from review

Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
This commit is contained in:
Lachlan Roberts 2022-02-08 15:59:23 +11:00
parent e1d149f84c
commit 5fe999caa2
6 changed files with 11 additions and 8 deletions

View File

@ -88,9 +88,12 @@ public class OpenIdConfiguration extends ContainerLifeCycle
* @param httpClient The {@link HttpClient} instance to use.
*/
public OpenIdConfiguration(@Name("issuer") String issuer,
@Name("authorizationEndpoint") String authorizationEndpoint, @Name("tokenEndpoint") String tokenEndpoint,
@Name("clientId") String clientId, @Name("clientSecret") String clientSecret,
@Name("authMethod") String authMethod, @Name("httpClient") HttpClient httpClient)
@Name("authorizationEndpoint") String authorizationEndpoint,
@Name("tokenEndpoint") String tokenEndpoint,
@Name("clientId") String clientId,
@Name("clientSecret") String clientSecret,
@Name("authMethod") String authMethod,
@Name("httpClient") HttpClient httpClient)
{
this.issuer = issuer;
this.clientId = clientId;

View File

@ -28,4 +28,4 @@ public class AdminPage extends HttpServlet
Map<String, Object> userInfo = (Map<String, Object>)request.getSession().getAttribute("org.eclipse.jetty.security.openid.claims");
response.getWriter().println(userInfo.get("sub") + ": success");
}
}
}

View File

@ -27,4 +27,4 @@ public class ErrorPage extends HttpServlet
response.getWriter().println("not authorized");
response.getWriter().println("<br><a href=\"" + request.getContextPath() + "\">Home</a>");
}
}
}

View File

@ -42,4 +42,4 @@ public class HomePage extends HttpServlet
response.getWriter().println("<br><a href=\"" + request.getContextPath() + "/login\">Login</a>");
}
}
}
}

View File

@ -27,4 +27,4 @@ public class LoginPage extends HttpServlet
response.getWriter().println("success");
response.getWriter().println("<br><a href=\"" + request.getContextPath() + "\">Home</a>");
}
}
}

View File

@ -26,4 +26,4 @@ public class LogoutPage extends HttpServlet
request.getSession().invalidate();
response.sendRedirect(request.getContextPath());
}
}
}