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

fixed get_peer function in misultin_utility #109

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fixed get_peer function in misultin_utility #109

wants to merge 1 commit into from

Conversation

vesperusiak
Copy link

Previously function could return IpTuple instead of {ok, IpTuple} in case when X-Forwarded-For header was present which conficted in other modules (in example case clause in 628 line in misultin_http)

Previously function could return IpTuple in case when X-Forwarded-For header was present which conficted in other modules (in example case clause in 628 line in misultin_http)
@ostinelli
Copy link
Owner

Hello,
Thank you for this PR. However, misultin has been discontinued some 3 years ago, as it can be seen from the README.

@vesperusiak
Copy link
Author

Oh, it's sad, because this server is much faster than 2 others, i think
that programmers should have choice between different libraries with
different assets.

On Fri, Apr 24, 2015 at 4:44 PM, Roberto Ostinelli <[email protected]

wrote:

Hello,
Thank you for this PR. However, misultin has been discontinued some 3
years ago, as it can be seen from the README
https://github.com/ostinelli/misultin/blob/master/README.md.


Reply to this email directly or view it on GitHub
#109 (comment).

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