Skip to content

connection, assistant: Use parsed port

Mike L. requested to merge CL-Jeremy/connections:master into master

Currently, the port is just ignored from the URL. That just doesn't work for me since my main use case is Xvnc.

This PR takes care of this, while also adding the necessary checks to allow scheme://host (defaulting to 5900/3389) but deny scheme://host:0+ (e. g. :0, :00, :00000) as well as scheme://host:65536 (and above) and URLs with path.

Merge request reports