Issue #7545 - changes from review

Signed-off-by: Lachlan Roberts <[email protected]>
(cherry picked from commit 5fe999c)

jetty-ee9/jetty-ee9-openid/src/main/java/org/eclipse/jetty/security/openid/OpenIdConfiguration.java

@@ -88,9 +88,12 @@ public OpenIdConfiguration(String issuer, String authorizationEndpoint, String t
      * @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;

tests/test-webapps/test-openid-webapp/src/main/java/org/eclipse/jetty/test/openid/AdminPage.java

@@ -28,4 +28,4 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
         Map<String, Object> userInfo = (Map<String, Object>)request.getSession().getAttribute("org.eclipse.jetty.security.openid.claims");
         response.getWriter().println(userInfo.get("sub") + ": success");
     }
-}
+}

tests/test-webapps/test-openid-webapp/src/main/java/org/eclipse/jetty/test/openid/ErrorPage.java

@@ -27,4 +27,4 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
         response.getWriter().println("not authorized");
         response.getWriter().println("<br><a href=\"" + request.getContextPath() + "\">Home</a>");
     }
-}
+}

tests/test-webapps/test-openid-webapp/src/main/java/org/eclipse/jetty/test/openid/HomePage.java

@@ -42,4 +42,4 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
             response.getWriter().println("<br><a href=\"" + request.getContextPath() + "/login\">Login</a>");
         }
     }
-}
+}

tests/test-webapps/test-openid-webapp/src/main/java/org/eclipse/jetty/test/openid/LoginPage.java

@@ -27,4 +27,4 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
         response.getWriter().println("success");
         response.getWriter().println("<br><a href=\"" + request.getContextPath() + "\">Home</a>");
     }
-}
+}

tests/test-webapps/test-openid-webapp/src/main/java/org/eclipse/jetty/test/openid/LogoutPage.java

@@ -26,4 +26,4 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response) t
         request.getSession().invalidate();
         response.sendRedirect(request.getContextPath());
     }
-}
+}