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

Remove lsquic entirely #4008

Merged
merged 2 commits into from
Aug 4, 2023
Merged

Remove lsquic entirely #4008

merged 2 commits into from
Aug 4, 2023

Conversation

syeopite
Copy link
Member

@syeopite syeopite commented Jul 26, 2023

Seems to work fine though I have no way of making sure the storyboard functionalities are kept since I can't test that on my local instance. It should be tested just in case.

Close #3752

@syeopite
Copy link
Member Author

Amoeba is broken on Crystal 1.9. See crystal-ameba/ameba#372. Once v1.5 of Ameba comes out it should be fixed.

Copy link
Member

@SamantazFox SamantazFox left a comment

Choose a reason for hiding this comment

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

Code looks good!

@SamantazFox SamantazFox added the need-testing This feature needs to be deployed and tested to see if it's working, and doesn't break something label Jul 28, 2023
@unixfox
Copy link
Member

unixfox commented Jul 28, 2023

I would like to merge this because I have been using invidious without quic compiled for months and this works great.

@syeopite
Copy link
Member Author

syeopite commented Aug 2, 2023

Safe to merge then?

@unixfox
Copy link
Member

unixfox commented Aug 2, 2023

Well that's what I wanted to do. Not sure about @SamantazFox

@unixfox unixfox merged commit 701f03d into iv-org:master Aug 4, 2023
7 of 8 checks passed
@unixfox
Copy link
Member

unixfox commented Aug 4, 2023

Merging as it's has been stable for a long time now.

@syeopite syeopite deleted the remove_lsquic branch August 6, 2023 19:50
@SamantazFox SamantazFox removed the need-testing This feature needs to be deployed and tested to see if it's working, and doesn't break something label Jan 14, 2024
999eagle added a commit to 999eagle/nixpkgs that referenced this pull request Feb 19, 2024
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.

[Enhancement] Force disable quic in the code
3 participants