From 3350ff967b8a803d6951114080bbefbc84c22b88 Mon Sep 17 00:00:00 2001 From: Suyash Garg Date: Tue, 29 Jan 2019 16:10:43 +0100 Subject: [PATCH] Use Optional.empty() --- .../nakadi/plugin/auth/DefaultAuthorizationService.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/src/main/java/org/zalando/nakadi/plugin/auth/DefaultAuthorizationService.java b/src/main/java/org/zalando/nakadi/plugin/auth/DefaultAuthorizationService.java index 04fd68072a..c69d19ddc7 100644 --- a/src/main/java/org/zalando/nakadi/plugin/auth/DefaultAuthorizationService.java +++ b/src/main/java/org/zalando/nakadi/plugin/auth/DefaultAuthorizationService.java @@ -1,6 +1,5 @@ package org.zalando.nakadi.plugin.auth; -import org.springframework.security.core.context.SecurityContextHolder; import org.zalando.nakadi.plugin.api.PluginException; import org.zalando.nakadi.plugin.api.authz.AuthorizationAttribute; import org.zalando.nakadi.plugin.api.authz.AuthorizationService; @@ -8,7 +7,6 @@ import org.zalando.nakadi.plugin.api.authz.Subject; import javax.annotation.Nullable; -import java.security.Principal; import java.util.List; import java.util.Optional; @@ -27,9 +25,7 @@ public boolean isAuthorizationAttributeValid(final AuthorizationAttribute author @Override @Nullable public Optional getSubject() { - return Optional.of( - () -> Optional.ofNullable(SecurityContextHolder.getContext().getAuthentication()) - .map(Principal::getName).orElse("unauthorized")); + return Optional.empty(); } @Override