Add authentication only check for @secured

CleverUnderDog · CleverUnderDog · commit 6d1b4b73fd0d · 2020-11-08T13:37:25.000+01:00
diff --git a/grpc-spring-boot-starter/src/main/java/org/lognet/springboot/grpc/security/GrpcServiceAuthorizationConfigurer.java b/grpc-spring-boot-starter/src/main/java/org/lognet/springboot/grpc/security/GrpcServiceAuthorizationConfigurer.java
@@ -74,8 +74,13 @@ public GrpcServiceAuthorizationConfigurer.Registry hasAnyRole(String... roles) {
         }
 
         public GrpcServiceAuthorizationConfigurer.Registry hasAnyAuthority(String... authorities) {
-            for (String auth : authorities) {
-                GrpcServiceAuthorizationConfigurer.this.registry.map(auth, methods);
+            if (authorities.length == 0) {
+                // Authenticate request
+                GrpcServiceAuthorizationConfigurer.this.registry.map(methods);
+            } else {
+                for (String auth : authorities) {
+                    GrpcServiceAuthorizationConfigurer.this.registry.map(auth, methods);
+                }
             }
             return GrpcServiceAuthorizationConfigurer.this.registry;
         }

Original file line number	Diff line number	Diff line change
`@@ -74,8 +74,13 @@ public GrpcServiceAuthorizationConfigurer.Registry hasAnyRole(String... roles) {`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`public GrpcServiceAuthorizationConfigurer.Registry hasAnyAuthority(String... authorities) {`
`77`		`- for (String auth : authorities) {`
`78`		`- GrpcServiceAuthorizationConfigurer.this.registry.map(auth, methods);`
	`77`	`+ if (authorities.length == 0) {`
	`78`	`+ // Authenticate request`
	`79`	`+ GrpcServiceAuthorizationConfigurer.this.registry.map(methods);`
	`80`	`+ } else {`
	`81`	`+ for (String auth : authorities) {`
	`82`	`+ GrpcServiceAuthorizationConfigurer.this.registry.map(auth, methods);`
	`83`	`+ }`
`79`	`84`	`}`
`80`	`85`	`return GrpcServiceAuthorizationConfigurer.this.registry;`
`81`	`86`	`}`