Exclude default ports 80/443 from base URL

[dinoboff]

Fix #59

Breaking changes: the URL is validated

[ddo]

too many changes. take me time to review all of them, be patient

[ddo]

this PR only "Exclude default ports 80/443 from base URL"? why take 13 files changes

[dinoboff]

@ddo Rebased on master

[dinoboff]

@ddo

Define parseUrl(url) with the other helpers.

[dinoboff]

That's crud fix. Maybe we should parse the URL without reg. exp. It would avoid any breaking changes.

[ddo]

i still checking the regex part. any source?

[dinoboff]

Based on https://en.wikipedia.org/wiki/URL (although I just notice the password part of the auth should be optional). The host is optional but required for OAuth request:

/^
  (https?):\/\/      # scheme://
  ([^:]+:[^@]+@)?    # [user[:password]@] (to fix)
  ([^:/?#]+)         # host
  (\:\d+)?           # [:port]
  (\/[^?#]*)?        # [/path]
  (\?[^#]*)?         # [?query]
  (#.*)?             # [#fragment] 
$/

[ddo]

i think we do it too complicated for this simple task?

[dinoboff]

Possibly. We don't need to extract the scheme + port and later the query with the same regexp.