reqwest icon indicating copy to clipboard operation
reqwest copied to clipboard
verified publisher icon seanmonstar

Add zstd support

Open paolobarbolini opened this issue 2 years ago • 6 comments

This PR has been recreated after I accidentally deleted my fork for #1464

Closes #1463

paolobarbolini avatar Jun 03 '23 11:06 paolobarbolini

This would be very useful indeed

ozgrakkurt avatar Nov 06 '23 00:11 ozgrakkurt

Is there any news / blocking point?

tafia avatar Nov 29 '23 01:11 tafia

Is there any news / blocking point?

No blockers on my part. I too wish we could get feedback on this PR, especially considering that lately there has been some progress with zstd on the web https://caniuse.com/zstd

paolobarbolini avatar Dec 04 '23 16:12 paolobarbolini

It looks like Chromium may be close to shipping it ^1. I'd be happy to rebase after the hyper v1 upgrade @seanmonstar.

paolobarbolini avatar Feb 05 '24 04:02 paolobarbolini

Thanks for pushing on this for so long. I remember looking a while ago, and it seemed to be unhappy on Windows. Assuming it works fine now, I'd be happy to merge.

seanmonstar avatar Mar 20 '24 16:03 seanmonstar

Yeah zstd had a Windows issue some time ago, it's been fixed upstream. I think this is ready for merging.

paolobarbolini avatar Mar 21 '24 08:03 paolobarbolini

Thanks again! I delayed merging a little bit just in case there was major problems with v0.12 that I'd need to revert or handle. But seems like its pretty smooth now. So this will go out in the next release.

seanmonstar avatar Apr 15 '24 14:04 seanmonstar