Problem details

From what I can tell, parsing https urls will work. This issue arises from specifying the default port when using a special scheme such as https. If a valid port other than 443 is specified when using https, then this should function as expected. It follows that you will get a similar problem when trying to parse http://www.google.com:80.

We are currently using url crate, based on the url standard. The standard stipulates:

If url’s port is url’s scheme’s default port, then set url’s port to null.

which sounds like the issue we're facing here.

Next steps

I wouldn't call this a bug and I do not think we need to update any documentation or code in Surreal as the url crate conforms to an existing standard. Users who have this problem can find this issue to work around it.

@LucasSovre @phughk

from surrealdb.

Great work @LivingLimes thank you!
servo/rust-url#214
As per the ticket, the solution would be to rely on port_or_known_default instead of port. So it is still a bug and that would be the solution. Thanks!

from surrealdb.

This is being worked on by @LivingLimes

from surrealdb.