search

Purify-net / PUrify

PUrify
Apache License 2.0
11 stars 6 forks source link

UriInfo class is improperly parsing URIs #36

Closed jweber closed 8 years ago

jweber commented 8 years ago

I initially made a pull request to the elasticsearch-net project to fix this but then realized that Purify was its own project.

This pull request is to primarily fix how URIs without query strings are parsed. It currently breaks with a System.ArgumentOutOfRangeException : StartIndex cannot be less than zero. exception.

It also handles removing the port number from the purified URI path.

Mpdreamz commented 8 years ago

TY this LGMT :+1: and thanks for taking the effort to open the PR here first, much appreciated!

glennblock commented 8 years ago

Thanks! On Tue, Dec 29, 2015 at 12:19 PM Martijn Laarman notifications@github.com wrote:

Merged #36 https://github.com/Purify-net/PUrify/pull/36.

— Reply to this email directly or view it on GitHub https://github.com/Purify-net/PUrify/pull/36#event-502371101.