Fix integration test failures by handling a NPE

wso2 · Jan 3, 2025 · 913e355 · 913e355
1 parent f78fbae
commit 913e355
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/.../org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/utils/APIUtil.java b/.../org.wso2.carbon.apimgt.impl/src/main/java/org/wso2/carbon/apimgt/impl/utils/APIUtil.java
@@ -5055,14 +5055,16 @@ public static Set<String> extractVisibleEnvironmentsForUser(List<Environment> en
                 String[] permittedRoles = environment.getVisibilityRoles();
                 if (permittedRoles != null && permittedRoles[0].equals("all")) {
                     environmentStringSet.add(environment.toString());
-                } else {
+                } else if (permittedRoles != null) {
                     for (String role : userRoles) {
                         for (String permission : permittedRoles) {
                             if (role.equals(permission)) {
                                 environmentStringSet.add(environment.toString());
                             }
                         }
                     }
+                } else {
+                    environmentStringSet.add(environment.toString());
                 }
             }
         }