Skip to content

Commit

Permalink
Remove unecessary file changes
Browse files Browse the repository at this point in the history
Signed-off-by: Derek Ho <[email protected]>
  • Loading branch information
derek-ho committed Feb 4, 2025
1 parent a1057ce commit b6aa196
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1026,11 +1026,7 @@ PrivilegesEvaluatorResponse providesExplicitPrivilege(
}
}
return PrivilegesEvaluatorResponse.insufficient(checkTable)
.reason(
resolvedIndices.getAllIndices().size() == 1
? "Insufficient permissions for the referenced index"
: "None of " + resolvedIndices.getAllIndices().size() + " referenced indices has sufficient permissions"
)
.reason("No explicit privileges have been provided for the referenced indices.")
.evaluationExceptions(exceptions);
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,7 +120,8 @@ public void testExtractCredentialsFailWhenTokenIsExpired() {
AuthCredentials ac = authenticator.extractCredentials(request, threadcontext);

assertNull("Should return null when JTI is expired", ac);
verify(log).debug(eq("Invalid or expired JWT token."), any(ExpiredJwtException.class));
verify(log).debug(eq("Invalid or expired api token."), any(ExpiredJwtException.class));

}

@Test
Expand Down Expand Up @@ -183,7 +184,6 @@ public void testAuthenticatorNotEnabled() {
Settings settings = Settings.builder()
.put("enabled", "false")
.put("signing_key", "U3VwZXJTZWNyZXRLZXlUaGF0SXNFeGFjdGx5NjRCeXRlc0xvbmdBbmRXaWxsV29ya1dpdGhIUzUxMkFsZ29yaXRobSEhCgo=")
.put("encryption_key", "MTIzNDU2Nzg5MDEyMzQ1Ng==")
.build();
ThreadContext threadContext = new ThreadContext(settings);

Expand Down

0 comments on commit b6aa196

Please sign in to comment.