Skip to content
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.

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

Update code for breaking change in werkzeug #1354

Merged
merged 1 commit into from Nov 6, 2023
Merged

Update code for breaking change in werkzeug #1354

merged 1 commit into from Nov 6, 2023

Conversation

dbalagansky
Copy link
Contributor

parse_authorization_header was removed here: pallets/werkzeug@bafffa9.

@tobias-urdin
Copy link
Contributor

Hello @dbalagansky thanks a lot for finding this breaking change that was blocking the upgrade jobs!

I've merged a pinning of werkzeug<3.0.0 on all stable branches to get CI on all branches working again and also
merged that on master branch with [1].

Please rebase this on master and unpin werkzeug completely (we don't pin unless it's on stable branches usually). I also think we need to workaround supporting werkzeug<3.0.0 on master as well so you'd need to add a try-except for this case.

Have a nice weekend!

[1] #1356

@dbalagansky dbalagansky changed the title Update code for breaking change in werkzeug and pin it Update code for breaking change in werkzeug Nov 6, 2023
@tobias-urdin tobias-urdin merged commit e7d5ab3 into gnocchixyz:master Nov 6, 2023
23 checks passed
@tobias-urdin
Copy link
Contributor

tobias-urdin commented Nov 6, 2023

Thanks!

openstack-mirroring pushed a commit to openstack/openstack that referenced this pull request Nov 7, 2023
* Update python-aodhclient from branch 'master'
  to 6dd743f31590ebb95b1acba5909d6bbabaa8df75
  - Fix gate
    
    Install gnocchi from master branch until the recent fixes
    will be included in next release.
    
    More info at:
    gnocchixyz/gnocchi#1354
    
    Change-Id: If5140dfb9d9b41d7f1044f653701bed5e33cab62
openstack-mirroring pushed a commit to openstack/python-aodhclient that referenced this pull request Nov 7, 2023
Install gnocchi from master branch until the recent fixes
will be included in next release.

More info at:
gnocchixyz/gnocchi#1354

Change-Id: If5140dfb9d9b41d7f1044f653701bed5e33cab62
@noonedeadpunk
Copy link

Folks, can we backport that to stable/4.6 ?

noonedeadpunk pushed a commit to noonedeadpunk/gnocchi that referenced this pull request Mar 14, 2024
This is the follow-up for PR gnocchixyz#1356
Cherry-picked from gnocchixyz#1354
@tobias-urdin
Copy link
Contributor

@Mergifyio backport stable/4.6

@tobias-urdin
Copy link
Contributor

@noonedeadpunk ^ I can look into backporting and release a new minor version with that

Copy link
Contributor

mergify bot commented Mar 15, 2024

backport stable/4.6

✅ Backports have been created

tobias-urdin added a commit that referenced this pull request Mar 15, 2024
Update code for breaking change in werkzeug (backport #1354)
@tobias-urdin
Copy link
Contributor

@noonedeadpunk 4.6.2 is released

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

Successfully merging this pull request may close these issues.

None yet

3 participants