Skip to content

Conversation

@burgesQ
Copy link

@burgesQ burgesQ commented Oct 27, 2025

  • the request' header can now be access via Conn.RequestHeaders() (thread safe)
  • the request' query can now be access via Conn.RequestQuery() (thread safe)
  • one may persist & access arbitrary data to the connection by using the Conn.SetData & Conn.GetData methods (see atomic.Pointer for hint about the whys)
  • the server' root URL can now be parametrized (was hard-coded to /socket.io/) (only std' http.Server ATM)

@burgesQ
Copy link
Author

burgesQ commented Oct 27, 2025

Seems I've wrongly committed some extra log messages. We should get rid of it pre-merge.

@burgesQ burgesQ force-pushed the main-q-stuffs branch 2 times, most recently from f4181ef to 7fdbd12 Compare October 28, 2025 14:56
@burgesQ burgesQ force-pushed the main-q-stuffs branch 3 times, most recently from 0aa8280 to 16175fe Compare October 28, 2025 15:02
The following payload would pass: `42/ssh,["resize",{"cols":123,"rows":41}]`
While this one would fail: `42/logview,["resize",{"cols":124,"rows":34}]`

Turns out some counter done on the sliced message did not take into
account already processed tokens
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.

1 participant