-
Notifications
You must be signed in to change notification settings - Fork 16
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.
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
Client IP #12
Comments
Casting to SocketWrapper seems a bit hacky, why not just add it to the interface? |
Yeah, that makes sense. Would be very nice info to have. Probably add to the env as servervariable style keys rather than "owin." prefixed - those values are only defined by spec. |
I've not read owin specs. I just was looking for a lightweight embeddable http server that matches my requirements. And would me nice to use original sources without making changes when I syncing sources |
Yeah, no need to worry about the owin spec. Lets add this - though it Louis - sent from mobile Reply to this email directly or view it on GitHub: |
Code review? See #13 |
In some cases it may usable to have client ip
Add to SocketWrapper
Change Go in Connection
Finally in Frame
The text was updated successfully, but these errors were encountered: