Use Optional.empty()

zalando · Jan 29, 2019 · 3350ff9 · 3350ff9
1 parent b297105
commit 3350ff9
Showing 1 changed file with 1 addition and 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
@@ -1,14 +1,12 @@
 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;
 import org.zalando.nakadi.plugin.api.authz.Resource;
 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<Subject> getSubject() {
-        return Optional.of(
-                () -> Optional.ofNullable(SecurityContextHolder.getContext().getAuthentication())
-                        .map(Principal::getName).orElse("unauthorized"));
+        return Optional.empty();
     }
 
     @Override