Kaz Yoshihara

Results 45 comments of Kaz Yoshihara

> Does this type by itself provide any use? I imagine you need to do more with it than simply materialize an opaque value from the map. This header is...

@ilammy I was waiting on a response from @seanmonstar for https://github.com/hyperium/headers/pull/88#discussion_r711390478. Once this is merged, I already have [a branch](https://github.com/kazk/tungstenite-rs/tree/permessage-deflate-rebased) using it, which is a rebased version of my original...

I'm currently working on this. So far, I've got it to pass some autobahn (no parameter support, so client passes 12, server passes both 12 and 13) by layering compression/decompression...

@application-developer-DA Thanks for reviewing! I'll start answering them soon. I opened this PR as a draft since it's not done (especially the client side like error handling, as you noticed),...

I was planning to continue after getting https://github.com/hyperium/headers/pull/88 merged (see https://github.com/snapview/tungstenite-rs/pull/235#discussion_r742305530), but forgot. I've been using this branch with a patched `warp` in production. Will try to take a look...

For now, I've rebased and cleaned up the commits. https://github.com/kazk/tungstenite-rs/tree/permessage-deflate-rebased I _might_ have some time over the weekend to continue.

Yeah, I'd like to keep that in `headers`. I'll update this PR to use `headers` (using my fork for now), and make sure it's useful for `tungstenite`. After making any...

I've updated to use `headers`. https://github.com/kazk/tungstenite-rs/commit/2e404a447c94eb05f3ee4059b7094bad06e20d1f I'll try to spend some time during the week to finish it and open a new PR.

@application-developer-DA The autobahn results don't match the expected in https://github.com/snapview/tungstenite-rs/tree/master/ for both server and client. ~~Is this a regression? I can open a separate PR to update if it just...