Skip to content

Unify HTTP/gRPC client resolution with OIDC progressive specificity cascade - #821

Merged
ricardozanini merged 1 commit into
quarkiverse:mainfrom
mcruzdev:issue-677
Aug 7, 2026
Merged

Unify HTTP/gRPC client resolution with OIDC progressive specificity cascade#821
ricardozanini merged 1 commit into
quarkiverse:mainfrom
mcruzdev:issue-677

Conversation

@mcruzdev

@mcruzdev mcruzdev commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Description

Fixes #677

Changes

  • Align HTTP with OIDC resolution
  • Align gRPC with OIDC resolution

Testing

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated (if applicable)
  • Tested manually (describe below if applicable)

Manual Testing

Checklist

Before submitting this PR, please ensure:

  • I ran the full build with integration tests locally: ./mvnw clean install -DskipITs=false
  • Code follows the project's code conventions
  • Tests have been added/updated to cover the changes
  • Documentation has been updated (if user-facing changes)
  • Commit messages are clear and follow conventional commits style
  • I have read and followed the Contributing Guide
  • I have read and comply with the LLM Usage Policy (if applicable)

Additional Notes

@mcruzdev
mcruzdev requested a review from a team as a code owner August 3, 2026 15:03
@github-actions

github-actions Bot commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown

🙈 The PR is closed and the preview is expired.

fjtirado
fjtirado reviewed Aug 3, 2026
View reviewed changes
Comment thread core/runtime/src/main/java/io/quarkiverse/flow/config/ClientConfigCascade.java Outdated
fjtirado
fjtirado approved these changes Aug 3, 2026
View reviewed changes

@fjtirado fjtirado left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Just picky comments

ricardozanini
ricardozanini reviewed Aug 4, 2026
View reviewed changes
Comment thread adr/2026-07-31-align-http-grpc-resolution-with-oidc.md Outdated
@mcruzdev
mcruzdev requested a review from ricardozanini August 4, 2026 20:58
ricardozanini
ricardozanini reviewed Aug 6, 2026
View reviewed changes

@ricardozanini ricardozanini left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please also add a migration guide 0.15.1 -> 0.16.0 explaining how the property changes for http and grpc.

Comment thread core/runtime/src/main/java/io/quarkiverse/flow/config/FlowHttpConfig.java Outdated
Comment thread core/runtime/src/main/java/io/quarkiverse/flow/config/FlowHttpConfig.java Outdated
Comment thread grpc/runtime/src/main/java/io/quarkiverse/flow/config/FlowGrpcConfig.java Outdated
This was referenced Aug 6, 2026
Closed
Merged
@ricardozanini

ricardozanini commented Aug 7, 2026

Copy link
Copy Markdown
Member

@mcruzdev can you rebase this and push again pleae?

@mcruzdev
Unify HTTP/gRPC client resolution with OIDC progressive specificity c…
30a840e 
…ascade

Signed-off-by: Matheus Cruz <matheuscruz.dev@gmail.com>
@ricardozanini
ricardozanini merged commit a55b7c8 into quarkiverse:main Aug 7, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ricardozanini ricardozanini ricardozanini approved these changes
@fjtirado fjtirado fjtirado approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Align gRPC Channel Resolution with HTTP Client Routing

3 participants

@mcruzdev @ricardozanini @fjtirado