druid icon indicating copy to clipboard operation
druid copied to clipboard

pacj4: add UserProfile attributes to AuthenticationResult context

Open jakubmatyszewski opened this issue 11 months ago • 3 comments

Description

I'm adding OIDC context to the AuthenticationResult returned by pac4j extension. I wanted to use this context as input in OpenPolicyAgent authorization. Since AuthenticationResult already accepts context as a parameter it felt okay to pass the profile attributes there.

Release note

pac4j: Add OIDC context to the authentication result


Key changed/added classes in this PR
  • org.apache.druid.security.pac4j.Pac4jFilter.doFilter()

This PR has:

  • [ ] been self-reviewed.
  • [ ] added documentation for new or modified features or behaviors.
  • [ ] a release note entry in the PR description.
  • [ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
  • [ ] added or updated version, license, or notice information in licenses.yaml
  • [ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
  • [ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
  • [ ] added integration tests.
  • [ ] been tested in a test Druid cluster.

jakubmatyszewski avatar Mar 12 '24 16:03 jakubmatyszewski