Revert RoutingContext#user() returning UserContext to User.

Motivation:

RoutingContext#user() method returns the io.vertx.ext.auth.User interface in Vert.x 4.x, its return type has been change to UserContext instead (in addition of other changes), as UserContext is the preferred API for interacting with the user and the User interface actually represents the User.

However, there is nothing wrong with having the user method returns the io.vertx.ext.auth.User interface, since the User interface is still the API for accessing user informations. The UserContext API can be returned from a userContext method instead. This avoids breaking the contract of the user method between 4.x and 5.0

Changes:

Introduce a new userContext method returning the UserContext and change the return type of the user method to return the User type.

Author: vietj

vertx-web-api-service/src/main/java/io/vertx/ext/web/api/service/impl/OpenAPIRouterHandlerImpl.java

@@ -54,7 +54,7 @@ public OpenAPIRouterHandlerImpl(EventBus eventBus, String address, DeliveryOptio
   protected Future<JsonObject> transformRequest(ValidatedRequest request, RoutingContext routingContext,
                                                 Operation operation) {
     JsonObject params = buildParametersObject(request);
-    JsonObject userPrincipal = Optional.ofNullable(routingContext.user().get()).map(User::principal).orElse(null);
+    JsonObject userPrincipal = Optional.ofNullable(routingContext.user()).map(User::principal).orElse(null);
 
     ServiceRequest sr = new ServiceRequest(
       params,

vertx-web-api-service/src/main/java/io/vertx/ext/web/api/service/impl/RouteToEBServiceHandlerImpl.java

@@ -73,7 +73,7 @@ public RouteToEBServiceHandlerImpl extraPayloadMapper(Function<RoutingContext, J
 
   private JsonObject buildPayload(RoutingContext context) {
     JsonObject params = context.get("parsedParameters") != null ? ((RequestParameters)context.get("parsedParameters")).toJson() : null;
-    User user = context.user().get();
+    User user = context.user();
     return new JsonObject().put("context", new ServiceRequest(
       params,
       context.request().headers(),

vertx-web-api-service/src/test/java/io/vertx/ext/web/api/service/tests/RouteToEBServiceHandlerTest.java

@@ -181,7 +181,7 @@ public void authorizedUserTest(Vertx vertx, VertxTestContext testContext) {
       .handler(
         ValidationHandlerBuilder.create(schemaRepo).build()
       ).handler(rc -> {
-        ((UserContextInternal) rc.user()).setUser(User.fromName("slinkydeveloper")); // Put user mock into context
+        ((UserContextInternal) rc.userContext()).setUser(User.fromName("slinkydeveloper")); // Put user mock into context
         rc.next();
       })
       .handler(

vertx-web-graphql/src/main/java/examples/GraphQLExamples.java

@@ -154,7 +154,7 @@ public void routingContextInDataFetchingEnvironment() {
 
       RoutingContext routingContext = environment.getGraphQlContext().get(RoutingContext.class);
 
-      UserContext user = routingContext.user();
+      UserContext user = routingContext.userContext();
 
       Future<List<Link>> future = retrieveLinksPostedBy(user);
       return future.toCompletionStage();

vertx-web-openapi-router/src/test/java/io/vertx/router/test/e2e/RouterBuilderSecurityOptionalCallbackawareTest.java

@@ -60,10 +60,10 @@ void testBuilderWithAuthn(VertxTestContext testContext) {
       .onSuccess(self -> {
         self
           .getRoute("opA")
-          .addHandler(ctx -> ctx.json(ctx.user().get().principal()));
+          .addHandler(ctx -> ctx.json(ctx.user().principal()));
         self
           .getRoute("opB")
-          .addHandler(ctx -> ctx.json(ctx.user().get().principal()));
+          .addHandler(ctx -> ctx.json(ctx.user().principal()));
       }))
       // this test may seem useless but it proves that the chain auth properly sets up a chain when the a handler
       // can perform redirects (callback aware) and doesn't throw an exception at setup time.

vertx-web-openapi-router/src/test/java/io/vertx/router/test/e2e/RouterBuilderSecurityOptionalTest.java

@@ -46,8 +46,8 @@ void testBuilderWithAuthn(VertxTestContext testContext) {
 
       rb.getRoute("pets")
         .addHandler(ctx -> {
-          if (ctx.user().authenticated()) {
-            ctx.json(ctx.user().get().principal());
+          if (ctx.userContext().authenticated()) {
+            ctx.json(ctx.user().principal());
           } else {
             ctx.json(null);
           }

vertx-web/src/main/asciidoc/index.adoc

@@ -1209,7 +1209,7 @@ make sure your authentication handler is before your application handlers on tho
 
 If the authentication handler has successfully authenticated the user it will inject a {@link io.vertx.ext.auth.User}
 object into the {@link io.vertx.ext.web.UserContext} so it's available in your handlers from the routing context:
-{@link io.vertx.ext.web.RoutingContext#user()}.
+{@link io.vertx.ext.web.RoutingContext#userContext()}.
 
 If you want your User object to be stored in the session so it's available between requests so you don't have to
 authenticate on each request, then you should make sure you have a session handler before the authentication handler.

vertx-web/src/main/java/examples/WebExamples.java

@@ -42,7 +42,6 @@
 import io.vertx.ext.web.sstore.SessionStore;
 
 import java.util.List;
-import java.util.function.Function;
 
 /**
  * These are the examples used in the documentation.
@@ -837,7 +836,7 @@ public void example38(Vertx vertx, AuthenticationProvider authProvider, Router r
       // This will require a login
 
       // This will have the value true
-      boolean isAuthenticated = ctx.user().authenticated();
+      boolean isAuthenticated = ctx.userContext().authenticated();
 
     });
   }
@@ -871,7 +870,7 @@ public void example39(Vertx vertx, AuthenticationProvider authProvider, Router r
         // This will require a login
 
         // This will have the value true
-        boolean isAuthenticated = ctx.user().authenticated();
+        boolean isAuthenticated = ctx.userContext().authenticated();
 
       });
 
@@ -1268,8 +1267,8 @@ public void example52(Vertx vertx) {
   public void example53(Vertx vertx) {
 
     Handler<RoutingContext> handler = ctx -> {
-      String theSubject = ctx.user().get().principal().getString("sub");
-      String someKey = ctx.user().get().principal().getString("someKey");
+      String theSubject = ctx.user().principal().getString("sub");
+      String someKey = ctx.user().principal().getString("someKey");
     };
   }
 
@@ -1495,7 +1494,7 @@ public void example62(Vertx vertx, Router router) {
       // at this moment your user object should contain the info
       // from the Oauth2 response, since this is a protected resource
       // as specified above in the handler config the user object is never null
-      User user = ctx.user().get();
+      User user = ctx.user();
       // just dump it to the client for demo purposes
       ctx.response().end(user.toString());
     });
@@ -1947,7 +1946,7 @@ public void example89(Router router) {
       .handler(ctx -> {
         // if the user isn't admin, we ask the user to login again as admin
         ctx
-          .user()
+          .userContext()
           .loginHint("admin")
           .impersonate();
       });
@@ -1958,7 +1957,7 @@ public void example90(Router router) {
       .route("/high/security/route/back/to/me")
       .handler(ctx -> {
         ctx
-          .user()
+          .userContext()
           .restore();
       });
   }
@@ -1968,7 +1967,7 @@ public void example91(Router router) {
       .route("/high/security/route/refresh/me")
       .handler(ctx -> {
         ctx
-          .user()
+          .userContext()
           .refresh();
       });
   }

vertx-web/src/main/java/io/vertx/ext/web/RoutingContext.java

@@ -23,6 +23,7 @@
 import io.vertx.core.internal.ContextInternal;
 import io.vertx.core.json.EncodeException;
 import io.vertx.core.json.Json;
+import io.vertx.ext.auth.User;
 import io.vertx.ext.web.impl.ParsableMIMEValue;
 import io.vertx.ext.web.impl.Utils;
 
@@ -223,7 +224,15 @@ public interface RoutingContext {
    * as perform authentication refreshes, logout and other operations.
    * @return the user context
    */
-  UserContext user();
+  UserContext userContext();
+
+  /**
+   * Get the authenticated user (if any). This will usually be injected by an auth handler if authentication if successful.
+   * @return  the user, or null if the current user is not authenticated.
+   */
+  default @Nullable User user() {
+    return userContext().get();
+  }
 
   /**
    * If the context is being routed to failure handlers after a failure has been triggered by calling

vertx-web/src/main/java/io/vertx/ext/web/handler/impl/AuthenticationHandlerImpl.java

@@ -60,7 +60,7 @@ public void handle(RoutingContext ctx) {
       ctx.request().pause();
     }
 
-    final User user = ctx.user().get();
+    final User user = ctx.user();
 
     if (user != null) {
       if (mfa != null) {
@@ -85,7 +85,7 @@ public void handle(RoutingContext ctx) {
     // perform the authentication
     authenticate(ctx)
       .onSuccess(authenticated -> {
-        ((UserContextInternal) ctx.user())
+        ((UserContextInternal) ctx.userContext())
           .setUser(authenticated);
         Session session = ctx.session();
         if (session != null) {