Fix port number extraction #80

Merged
merged 1 commit into from Mar 16, 2014

Conversation

Projects
None yet
2 participants
Contributor

AlexanderPavlenko commented Feb 19, 2013

fixes #79
basically, two things to say:

  1. looks like it was comparing ':' position from proxy's url to '/' position from request's url, that totally has no sense to me
  2. in case when no proxy used, valid url should not use colon without escaping in path segment anyway
AlexanderPavlenko fix port extraction 1c55dd6

@jkramer jkramer pushed a commit that referenced this pull request Mar 16, 2014

Jonas Kramer Merge pull request #80 from AlexanderPavlenko/master
Fix port number extraction
3e37d10

@jkramer jkramer merged commit 3e37d10 into jkramer:master Mar 16, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment