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.

Sign up for GitHub

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

Jump to bottom

Basic upgrade implementation for WebSocket #63

Closed
wants to merge 6 commits into from
Closed

Basic upgrade implementation for WebSocket #63

wants to merge 6 commits into from

Conversation

bigbes
Copy link
Contributor

@bigbes bigbes commented Sep 11, 2018

  • Little dependent refactoring
  • Introduced session object (stores socket and all needed information for connection), probably better to rename it, but dunno
  • Also move some functions to global (needed by other modules)

Closes gh-62

@bigbes
Move http server to metatable
f135b3a
@bigbes
Introduce session object
c2f0895 
Session object will store connection information (protocol, peer, socket)
@bigbes
Extract all request parsing/creation to separate function
d3a11ce 
Also rename all http_server_ to httpd (old naming convention)
@bigbes
Initial upgrade implementation
b69cf36
@bigbes
Allow callbacks to be callable tables also
20abe16
@bigbes
Add configuration option "ignore_unknown_upgrade"
14d17f3 
In case when upgrade should be ignored
@bigbes
Copy link
Contributor Author

bigbes commented Apr 18, 2019

@filonenko-mikhail ?

@Satbek Satbek closed this Oct 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@filonenko-mikhail filonenko-mikhail filonenko-mikhail left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Basic 'upgrade' implementation
3 participants
@bigbes @filonenko-mikhail @Satbek