Skip to content

Downgrade to our fork of http2#3141

Merged
mdimjasevic merged 3 commits intodevelopfrom
pcapriotti/revert-http2-upgrade
Mar 13, 2023
Merged

Downgrade to our fork of http2#3141
mdimjasevic merged 3 commits intodevelopfrom
pcapriotti/revert-http2-upgrade

Conversation

@pcapriotti
Copy link
Contributor

@pcapriotti pcapriotti commented Mar 9, 2023

Downgrade http2 library.

https://wearezeta.atlassian.net/browse/FS-1609

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

It seems the released version of http2 is causing issues with streaming
of assets via federator.
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Mar 9, 2023
@mdimjasevic mdimjasevic merged commit 02ba744 into develop Mar 13, 2023
@mdimjasevic mdimjasevic deleted the pcapriotti/revert-http2-upgrade branch March 13, 2023 09:35
mdimjasevic pushed a commit that referenced this pull request Mar 13, 2023
* Downgrade to our fork of http2

It seems the released version of http2 is causing issues with streaming
of assets via federator.

* Add CHANGELOG entry
battermann pushed a commit that referenced this pull request Mar 15, 2023
* Downgrade to our fork of http2

It seems the released version of http2 is causing issues with streaming
of assets via federator.

* Add CHANGELOG entry
akshaymankar added a commit that referenced this pull request Mar 29, 2023
This reverts commit 02ba744.

The real issue was resolved in #3154
akshaymankar added a commit that referenced this pull request Mar 29, 2023
This reverts commit 02ba744.

The real issue was resolved in #3154
lepsa pushed a commit to lepsa/wire-server that referenced this pull request Nov 28, 2023
* Downgrade to our fork of http2

It seems the released version of http2 is causing issues with streaming
of assets via federator.

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

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

Comments