Skip to content

Conversation

@gabriel-farache
Copy link
Contributor

@gabriel-farache gabriel-farache commented Jun 19, 2025

Fixes #1210
Many thanks for submitting your Pull Request ❤️!

Please make sure that your PR meets the following requirements:

  • You have read the contributors guide
  • Your code is properly formatted according to our code style
  • Pull Request title contains the target branch if not targeting main: [0.9.x] Subject
  • Pull Request contains link to the issue
  • Pull Request contains link to any dependent or related Pull Request
  • Pull Request contains description of the issue
  • Pull Request does not include fixes for issues other than the main ticket
How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-quarkus2 to backport the original PR to the quarkus2 branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.

If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@gabriel-farache gabriel-farache requested a review from a team as a code owner June 19, 2025 09:51
…te to avoid immutable error

Signed-off-by: gabriel-farache <[email protected]>
@gabriel-farache
Copy link
Contributor Author

cc @wmedvede @fjtirado
@ricardozanini this should be part of the next release too

@gabriel-farache
Copy link
Contributor Author

gabriel-farache commented Jun 19, 2025

Copy link
Contributor

@fjtirado fjtirado left a comment

Choose a reason for hiding this comment

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

@ricardozanini This completes #1189
Needs to be included in the upcoming release

Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Thanks @gabriel-farache! Please add a test case to validate the changes.

@ricardozanini ricardozanini added area:client This item is related to the client extension backport-main-lts DO NOT MERGE labels Jun 19, 2025
Signed-off-by: gabriel-farache <[email protected]>
@gabriel-farache
Copy link
Contributor Author

Thanks @gabriel-farache! Please add a test case to validate the changes.

@ricardozanini Done

@ricardozanini ricardozanini merged commit ad9c3a9 into quarkiverse:main Jun 19, 2025
11 checks passed
@ricardozanini
Copy link
Member

ricardozanini commented Jun 19, 2025

@hbelmiro, do you have any clues as to why @gabriel-farache's PRs are not being included in the backport automation? 🤔

@gabriel-farache can you please manually cherry-pick this PR to main-lts?

@ricardozanini
Copy link
Member

@gabriel-farache forget it, looks like CI is just lazy today: https://github.com/quarkiverse/quarkus-openapi-generator/actions/runs/15759894794

Keep an eye on it, though. If it fails, we might have to push it manually.

github-actions bot pushed a commit that referenced this pull request Jun 19, 2025
… immutable error (#1211)

* Fix #1210 - use put method to set headers in oauth2 delegate to avoid immutable error

Signed-off-by: gabriel-farache <[email protected]>

* Add unit test

Signed-off-by: gabriel-farache <[email protected]>

---------

Signed-off-by: gabriel-farache <[email protected]>
ricardozanini pushed a commit that referenced this pull request Jun 19, 2025
… immutable error (#1211) (#1213)

* Fix #1210 - use put method to set headers in oauth2 delegate to avoid immutable error



* Add unit test



---------

Signed-off-by: gabriel-farache <[email protected]>
Co-authored-by: gabriel-farache <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:client This item is related to the client extension backport-main-lts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OAUTH2 delegates throws java.lang.UnsupportedOperationException when adding the token to the request header parameters

3 participants