Change to use copyof in getSecurityRoles in AuthCredentials class

RyanL1997 · RyanL1997 · commit a805843a0bc1 · 2023-08-25T11:11:13.000-07:00
Signed-off-by: Ryan Liang &lt;jiallian@amazon.com&gt;
diff --git a/src/main/java/org/opensearch/security/user/AuthCredentials.java b/src/main/java/org/opensearch/security/user/AuthCredentials.java
@@ -222,7 +222,7 @@ public Set<String> getBackendRoles() {
      * @return Defensive copy of the security roles this user is member of.
      */
     public Set<String> getSecurityRoles() {
-        return new HashSet<String>(securityRoles);
+        return Set.copyOf(securityRoles);
     }
 
     public boolean isComplete() {

Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,7 @@ public Set<String> getBackendRoles() {`
`222`	`222`	`* @return Defensive copy of the security roles this user is member of.`
`223`	`223`	`*/`
`224`	`224`	`public Set<String> getSecurityRoles() {`
`225`		`- return new HashSet<String>(securityRoles);`
	`225`	`+ return Set.copyOf(securityRoles);`
`226`	`226`	`}`
`227`	`227`
`228`	`228`	`public boolean isComplete() {`