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

Pin UI service dependencies due to aiohttp websocket regression #173

Merged
merged 1 commit into from
May 5, 2021

Conversation

msavela
Copy link
Contributor

@msavela msavela commented May 5, 2021

Revert recent changes on UI service requirements.txt due to aiohttp 3.7 websocket regression.

There are multiple reported issues on websocket behaviour:

Let's keep these dependencies pinned for now and upgrade each of these in controlled manner later on so we can resume realtime functionality.

@msavela msavela requested review from ferras and saikonen May 5, 2021 10:23
@saikonen saikonen merged commit c58f383 into ui May 5, 2021
@saikonen saikonen deleted the fix/ui-requirements-pinned branch May 5, 2021 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants