Skip to content

[3.33] 3.33.4 backports 4 - #56511

Open
jmartisk wants to merge 7 commits into
quarkusio:3.33from
jmartisk:3.33.4-backports-4
Open

[3.33] 3.33.4 backports 4#56511
jmartisk wants to merge 7 commits into
quarkusio:3.33from
jmartisk:3.33.4-backports-4

Conversation

@jmartisk

@jmartisk jmartisk commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@jmartisk
jmartisk requested a review from a team September 9, 2026 12:20
@quarkus-bot

quarkus-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

Thanks for your pull request!

Your pull request does not follow our editorial rules. Could you have a look?

  • description should not be empty, describe your intent or provide links to the issues this PR is fixing (using Fixes #NNNNN) or changelogs

This message is automatically generated by a bot.

@quarkus-bot

quarkus-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

/cc @aloubyansky (3.33), @gsmet (3.33), @rsvoboda (3.33)

@quarkus-bot quarkus-bot Bot added area/dependencies Pull requests that update a dependency file area/documentation area/maven area/oidc labels Sep 9, 2026
@quarkus-bot quarkus-bot Bot changed the title 3.33.4 backports 4 [3.33] 3.33.4 backports 4 Sep 9, 2026
@quarkus-bot quarkus-bot Bot added area/cli Related to quarkus cli (not maven/gradle/etc.) area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/gradle Gradle area/rest area/rest-client labels Sep 9, 2026
@quarkus-bot

This comment has been minimized.

cnbehr and others added 7 commits September 9, 2026 14:40
An empty or delimiter-only bearer token has zero StringTokenizer tokens, so the
unguarded nextToken() in getJwtContentPart threw NoSuchElementException. That is
not an AuthenticationFailedException, so it escaped
BearerAuthenticationMechanism and AbstractHttpAuthorizer turned it into
routingContext.fail(t) -- HTTP 500 plus an ERROR stack trace, remotely
triggerable by any unauthenticated caller on any bearer-protected endpoint.

The method already returns null for 'not a JWT', so hoisting a hasMoreTokens()
check above the first nextToken() needs no caller changes: isOpaqueToken
correctly returns true and the request falls through to normal 401 handling.

decodeJwtHeaders and decodeJwtHeadersAsString in OidcUtils carried the identical
unguarded call and are fixed the same way.

Fixes quarkusio#55887

(cherry picked from commit 56247e9)
Signed-off-by: Vaibhav Srivastava <vaibhavsri1712@gmail.com>
(cherry picked from commit 7768746)
(cherry picked from commit 7fc6601)
Bumps [org.jsoup:jsoup](https://github.com/jhy/jsoup) from 1.23.1 to 1.23.2.
- [Release notes](https://github.com/jhy/jsoup/releases)
- [Changelog](https://github.com/jhy/jsoup/blob/master/CHANGES.md)
- [Commits](jhy/jsoup@jsoup-1.23.1...jsoup-1.23.2)

---
updated-dependencies:
- dependency-name: org.jsoup:jsoup
  dependency-version: 1.23.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 8a4538b)
We see the occasional timeout being reached in our Mandrel CI which
tests graal/master builds (GraalVM CE Innovation builds). What's more
the CI runs on free GHA runners, with 4 CPU cores and 14GB of memory.
Those builds received features which increase build time.

For example:

- Priority inliner (approx. 30-40% longer build times)
  oracle/graal#14138

Some successful builds take around up to 2 minutes 50-ish seconds to
complete for the said native IT test. We only see failures of the test
after this upstream PR merged. This seems in-line with the build time
increase.  Note that older builds of graal/master from July 2026 took
about 2 minutes and 30 seconds at most.

(cherry picked from commit e57529f)
@quarkus-bot

quarkus-bot Bot commented Sep 9, 2026

Copy link
Copy Markdown

Status for workflow Quarkus Documentation CI

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

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

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

🎊 PR Preview d4ff032 has been successfully built and deployed to https://quarkus-pr-main-56511-preview.surge.sh/version/main/guides/

  • Images of blog posts older than 3 months are not available.
  • Newsletters older than 3 months are not available.

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

Labels

area/cli Related to quarkus cli (not maven/gradle/etc.) area/dependencies Pull requests that update a dependency file area/devtools Issues/PR related to maven, gradle, platform and cli tooling/plugins area/documentation area/gradle Gradle area/maven area/oidc area/rest area/rest-client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants