Skip to content

Always return initialized TokenIntrospection from the CDI producer #49022

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 21, 2025
Merged

Always return initialized TokenIntrospection from the CDI producer #49022

merged 1 commit into from
Jul 21, 2025

Conversation

sberyozkin
Copy link
Member

@sberyozkin sberyozkin commented Jul 21, 2025 •
edited
Loading

Avoid CDI producer exception when the token introspection is null, as reported by @sschellh on Zulip

@sberyozkin sberyozkin requested a review from michalvavrik July 21, 2025 14:39
michalvavrik
michalvavrik approved these changes Jul 21, 2025
View reviewed changes
Copy link
Member

@michalvavrik michalvavrik left a comment

Choose a reason for hiding this comment

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

+1

@michalvavrik michalvavrik requested a review from gastaldi July 21, 2025 15:16
@quarkus-bot quarkus-bot
Copy link

quarkus-bot bot commented Jul 21, 2025 •
edited by github-actions bot
Loading

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit b548da8.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

Flaky tests - Develocity

⚙️ Maven Tests - JDK 17 Windows

📦 integration-tests/devmode

✖ io.quarkus.test.devui.DevUIGrpcSmokeTest.testTestService - History

  • Too many recursions, message not returned for id [1748283692] - java.lang.RuntimeException
java.lang.RuntimeException: Too many recursions, message not returned for id [1748283692]
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:168)
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:171)
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:171)
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:171)
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:171)
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:171)
	at io.quarkus.devui.tests.DevUIJsonRPCTest.objectResultFromJsonRPC(DevUIJsonRPCTest.java:171)

@sberyozkin sberyozkin merged commit a5feeac into quarkusio:main Jul 21, 2025
28 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.26 - main milestone Jul 21, 2025
@gsmet gsmet modified the milestones: 3.26 - main, 3.24.5 Jul 22, 2025
@jmartisk jmartisk modified the milestones: 3.24.5, 3.20.3 Aug 27, 2025
@jmartisk jmartisk moved this to 3.20.3 in Backports for 3.20 Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gastaldi gastaldi gastaldi approved these changes

+1 more reviewer

@michalvavrik michalvavrik michalvavrik approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area/oidc triage/flaky-test

Projects

Backports for 3.20
Status: 3.20.3

Milestone

3.20.3

Development

Successfully merging this pull request may close these issues.

5 participants

@sberyozkin @gastaldi @michalvavrik @jmartisk @gsmet