kfaraz commented on code in PR #18688:
URL: https://github.com/apache/druid/pull/18688#discussion_r2454538052
##########
extensions-core/druid-basic-security/src/main/java/org/apache/druid/security/basic/authorization/endpoint/BasicAuthorizerResource.java:
##########
@@ -610,6 +612,24 @@ public Response getCachedSerializedGroupMap(
return resourceHandler.getCachedGroupMappingMaps(authorizerName);
}
+ @GET
+ @Path("/db/{authorizerName}/history")
+ @Produces(MediaType.APPLICATION_JSON)
+ @ResourceFilters(BasicSecurityResourceFilter.class)
+ public Response getAuthorizerUpdateHistory(
Review Comment:
Sure, we can leave out the API. It just seemed weird to have audit logs and
no way to see them for the end user. But we can add it later when needed.
For now, I have updated the embedded test so that it fetches the audit
entries from the DB directly.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]