Skip to content

chore: default authentication path to /* in platform-http-main#22474

Open
Croway wants to merge 1 commit intoapache:mainfrom
Croway:chore/platform-http-auth-path-default
Open

chore: default authentication path to /* in platform-http-main#22474
Croway wants to merge 1 commit intoapache:mainfrom
Croway:chore/platform-http-auth-path-default

Conversation

@Croway
Copy link
Copy Markdown
Contributor

@Croway Croway commented Apr 7, 2026

Summary

  • Default the authentication path to /* when not explicitly configured in BasicAuthenticationConfigurer and JWTAuthenticationConfigurer
  • Extract shared path resolution logic into MainAuthenticationConfigurer default method
  • Add tests for non-root context path and selective path authentication
  • Add upgrade guide entry

Test plan

  • Existing auth tests pass (Basic + JWT)
  • New test: non-root context path with implicit auth path
  • New test: explicit authenticationPath for selective protection

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

🌟 Thank you for your contribution to the Apache Camel project! 🌟
🤖 CI automation will test this PR automatically.

🐫 Apache Camel Committers, please review the following items:

  • First-time contributors require MANUAL approval for the GitHub Actions to run
  • You can use the command /component-test (camel-)component-name1 (camel-)component-name2.. to request a test from the test bot although they are normally detected and executed by CI.
  • You can label PRs using build-all, build-dependents, skip-tests and test-dependents to fine-tune the checks executed by this PR.
  • Build and test logs are available in the summary page. Only Apache Camel committers have access to the summary.

⚠️ Be careful when sharing logs. Review their contents before sharing them publicly.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants