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.

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

Interface to get scheme used in request #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jadahl
Copy link

@jadahl jadahl commented Feb 16, 2011

(copied from previous pull request)

It could be useful to know whether for example https or http (or any other protocol) were used in the request to nitrogen. To know this, there needs to be an interface for this in simple_bridge. This commit contains a proposed interface, and an implementation in yaws. If interested, I could help out in implementing support in the other bridge implementations as well.

Implemented support for 'scheme' in yaws request bridge.
RequestBridge:port() returns the local port of the request.
Implemented support for 'port' in yaws request bridge.
@choptastic
Copy link
Member

It is my intention to re-introduce SSL to simple_bridge 2.0, and then will re-add the scheme() to finally make this pull request be resolved.

Let's hear it for 4-year-old pull requests!

choptastic pushed a commit that referenced this pull request Sep 26, 2018
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.

2 participants