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

Implement more ContentEncoding options #116

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

daxpedda
Copy link

@daxpedda daxpedda commented Aug 6, 2022

Only gzip was implemented for ContentEncoding, I implemented all other known encodings. See https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Content-Encoding.

@daxpedda
Copy link
Author

daxpedda commented Aug 6, 2022

I'm pretty sure this CI failure has nothing to do with this PR, right?

@daxpedda daxpedda closed this Sep 3, 2022
@daxpedda daxpedda reopened this Sep 3, 2022
@daxpedda
Copy link
Author

daxpedda commented Sep 3, 2022

I tried restarting CI, but got stuck at "workflow awaiting approval" again.

@daxpedda daxpedda requested a review from seanmonstar September 3, 2022 13:33
@daxpedda
Copy link
Author

The log gets automatically deleted until I look at the PR by chance again, there is no notification or anything, so I don't know whats going on. When I test the MSRV locally everything is fine.

I would appreciate if somebody restarts the MSRV test again and pings me so I can actually take a look at the logs.
I'm still assuming that this is a CI issue.

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