Skip to content

Conversation

@PoltoS
Copy link

@PoltoS PoltoS commented Apr 30, 2025

Partially reverts ef4a85d

Closes #3367

@PoltoS
Copy link
Author

PoltoS commented Jun 14, 2025

Hi @lws-team , could you please check this fix. The reasoning is in #3367

@lws-team
Copy link
Member

Yes I get it that something is up with this, however the code runs in different modes which act differently depending on if you have pm_deflate (and perhaps ws-over-h2) or not. I need to find the time to figure out the scope of it and try them all.

@roxlu
Copy link

roxlu commented Jul 27, 2025

Thanks @PoltoS and @lws-team, would be great if this gets fixed.

@lws-team
Copy link
Member

I can't reproduce it, as told here: #3367 (comment) Instructions for the case where there's a problem will help me solve it (whatever it is).

@PoltoS
Copy link
Author

PoltoS commented Jul 27, 2025

@lws-team In #3367 I shared my compilation parameters that lead to the issue with the current code. Doesn't it work for you to reproduce the issue? If not, please share on which system do you work? Maybe you can add this combination in CI/CD to check?

@lws-team lws-team force-pushed the main branch 5 times, most recently from efb9f35 to 1faedc3 Compare September 6, 2025 09:49
@lws-team lws-team force-pushed the main branch 8 times, most recently from 721f91f to 49de3d2 Compare September 15, 2025 14:00
@lws-team lws-team force-pushed the main branch 6 times, most recently from a800d4a to 7f2f518 Compare September 18, 2025 08:03
@lws-team lws-team force-pushed the main branch 3 times, most recently from baf4897 to 0c7fa23 Compare October 1, 2025 14:55
@lws-team lws-team force-pushed the main branch 3 times, most recently from 2b74af4 to 5f77374 Compare October 12, 2025 10:02
@lws-team lws-team force-pushed the main branch 5 times, most recently from a677221 to 0c67054 Compare October 24, 2025 16:36
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.

Connection to WS server fails

3 participants