Skip to content

Commit

Permalink
use non-deprecated authenticate methods (hyperledger#5852)
Browse files Browse the repository at this point in the history
Signed-off-by: Sally MacFarlane <macfarla.github@gmail.com>
  • Loading branch information
macfarla authored and NickSneo committed Nov 12, 2023
1 parent e7f272c commit 7d13285
Showing 1 changed file with 8 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -33,6 +33,9 @@
import io.vertx.ext.auth.JWTOptions;
import io.vertx.ext.auth.User;
import io.vertx.ext.auth.authentication.AuthenticationProvider;
import io.vertx.ext.auth.authentication.Credentials;
import io.vertx.ext.auth.authentication.TokenCredentials;
import io.vertx.ext.auth.authentication.UsernamePasswordCredentials;
import io.vertx.ext.auth.jwt.JWTAuth;
import io.vertx.ext.auth.jwt.JWTAuthOptions;
import io.vertx.ext.web.RoutingContext;
Expand Down Expand Up @@ -166,7 +169,7 @@ public void handleLogin(final RoutingContext routingContext) {

private void login(
final RoutingContext routingContext, final AuthenticationProvider credentialAuthProvider) {
final JsonObject requestBody = routingContext.getBodyAsJson();
final JsonObject requestBody = routingContext.body().asJsonObject();

if (requestBody == null) {
routingContext
Expand All @@ -181,8 +184,10 @@ private void login(
final JsonObject authParams = new JsonObject();
authParams.put(USERNAME, requestBody.getValue(USERNAME));
authParams.put("password", requestBody.getValue("password"));
final Credentials credentials = new UsernamePasswordCredentials(authParams);

credentialAuthProvider.authenticate(
authParams,
credentials,
r -> {
if (r.failed()) {
routingContext
Expand Down Expand Up @@ -227,7 +232,7 @@ public void authenticate(final String token, final Handler<Optional<User>> handl
try {
getJwtAuthProvider()
.authenticate(
new JsonObject().put("token", token),
new TokenCredentials(new JsonObject().put("token", token)),
r -> {
if (r.succeeded()) {
final Optional<User> user = Optional.ofNullable(r.result());
Expand Down

0 comments on commit 7d13285

Please sign in to comment.