Change Record._endpoint_from_url() to handle stripped http/https default ports

[markkuleinio]

Fixes #309 (that was caused by the parsing change in #308)

NetBox removes the default http/https (80 or 443 respectively) port from the response object URL even though it is given for any reason in the pynetbox.Api instantiation. That makes Record._endpoint_from_url() generate incorrect App names (it removes too many characters from the given URL) in those cases.

This PR makes Record._endpoint_from_url() to remove the default ports so that the App name is correctly parsed. Parsing does not look very elegant but works 😐

[markkuleinio]

Or, should we just remove the default ports from the URL right in the pynetbox.Api instantiation and forget all this mangling in Record._endpoint_from_url()?

[zachmoody]

Or, should we just remove the default ports from the URL right in the pynetbox.Api instantiation and forget all this mangling in Record._endpoint_from_url()?

Yeah, if we're going to take on sanitizing input, we should probably do it there. I can whip up a PR for it a bit later.

[markkuleinio]

Ok, I'll close this (https://tools.ietf.org/html/rfc3986#section-3.2.3 was mentioned in #309).