Skip to content

ATLAS-5385: Rename header authentication config to atlas.authentication.method.header.* - #735

Merged
nixonrodrigues merged 5 commits into
apache:masterfrom
ramackri:header-auth-config-rename
Aug 20, 2026
Merged

ATLAS-5385: Rename header authentication config to atlas.authentication.method.header.*#735
nixonrodrigues merged 5 commits into
apache:masterfrom
ramackri:header-auth-config-rename

Conversation

@ramackri

@ramackri ramackri commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Summary

Rename header authentication properties from atlas.authn.header.* to atlas.authentication.method.header.* in the existing AtlasHeaderPreAuthFilter. No behavior change; same filter logic as master.

Migration

# Before
atlas.authn.header.enabled=true
atlas.authn.header.username=<header-name>
atlas.authn.header.roles=<header-name>
atlas.authn.header.requestid=<header-name>

# After
atlas.authentication.method.header.enabled=true
atlas.authentication.method.header.username=<header-name>
atlas.authentication.method.header.roles=<header-name>
atlas.authentication.method.header.request-id=<header-name>

Test plan

  • Enable header auth with updated property keys in atlas-application.properties
  • Verify API access with curl using configured identity headers
  • Run AtlasHeaderPreAuthFilterTest

https://issues.apache.org/jira/browse/ATLAS-5385

…on.method.header.*

Replace legacy atlas.authn.header.* properties with a generic logical-key to HTTP-header mapping scheme so deployers can configure any proxy header names without vendor-specific defaults in OSS code.
@ramackri
ramackri force-pushed the header-auth-config-rename branch from d8057b2 to 50a3ca3 Compare August 19, 2026 10:02
Use server-common User class in AtlasHeaderPreAuthFilter and accept only
documented property suffixes (username, roles, request-id).
Drop AtlasHeaderAuthConfiguration and keep the existing filter logic.
Update atlas.authn.header.* properties to atlas.authentication.method.header.*.
Comment thread distro/src/conf/atlas-application.properties Outdated
…tings

Address review feedback to colocate header auth examples under Security
Properties alongside LDAP and AD configuration.
Comment thread distro/src/conf/atlas-application.properties Outdated
Align enable flag with other atlas.authentication.method.* settings.

@nixonrodrigues nixonrodrigues left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@nixonrodrigues
nixonrodrigues merged commit bcfe1a2 into apache:master Aug 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants