Skip to content

CAMEL-21922 - Add support for adding OAuth2 scope as query or form parameter#17634

Closed
Jefwillems wants to merge 2 commits intoapache:mainfrom
Jefwillems:main
Closed

CAMEL-21922 - Add support for adding OAuth2 scope as query or form parameter#17634
Jefwillems wants to merge 2 commits intoapache:mainfrom
Jefwillems:main

Conversation

@Jefwillems
Copy link

@Jefwillems Jefwillems commented Apr 1, 2025

Description

This pull request adds an option to add the oauth scope as either a query param (default, so it doesn't break anything) or as a form parameter. When testing the current implementation we noticed EntraId not picking up the scope when added as a query param. We ended up writing our own HttpClientConfigurer to add it in the form, but this might be cleaner. documentation about this can be found here: https://learn.microsoft.com/en-us/entra/identity-platform/v2-oauth2-client-creds-grant-flow#first-case-access-token-request-with-a-shared-secret

Target

  • I checked that the commit is targeting the correct branch (Camel 4 uses the main branch)

Tracking

  • If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue CAMEL-21922 filed for the change (usually before you start working on it).

Apache Camel coding standards and style

  • I checked that each commit in the pull request has a meaningful subject line and body.

  • I have run mvn clean install -DskipTests locally from root folder and I have committed all auto-generated changes.

@github-actions
Copy link
Contributor

github-actions bot commented Apr 1, 2025

🌟 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.

  • 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.

@Jefwillems
Copy link
Author

This seems to already exist in another PR. I just didn't find the JIRA issue. closing now

@Jefwillems Jefwillems closed this Apr 1, 2025
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.

1 participant