Replace null check with has value check because proto fields can neve…

…r be null.
kannanjgithub · Nov 8, 2024 · 62c2557 · 62c2557
1 parent d6c8029
commit 62c2557
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/xds/src/main/java/io/grpc/xds/GcpAuthenticationFilter.java b/xds/src/main/java/io/grpc/xds/GcpAuthenticationFilter.java
@@ -72,8 +72,8 @@ public ConfigOrError<? extends FilterConfig> parseFilterConfig(Message rawProtoM
 
     long cacheSize = 10;
     // Validate cache_config
-    TokenCacheConfig cacheConfig = gcpAuthnProto.getCacheConfig();
-    if (cacheConfig != null) {
+    if (gcpAuthnProto.hasCacheConfig()) {
+      TokenCacheConfig cacheConfig = gcpAuthnProto.getCacheConfig();
       cacheSize = cacheConfig.getCacheSize().getValue();
       if (cacheSize == 0) {
         return ConfigOrError.fromError(