Closed verdverm closed 6 years ago
Added some docs; also removed one line that was a duplicate
@verdverm https
shouldn't automatically mean 443 port. It means port 443
but if port is omitted from the string. Kinda the same as http
and 80
. I think this part of pull request should be amended
@vlasenko I was thinking about that as I wrote it. Cleaned it up, if protocol and port match (for both http(s)), then it will omit it for aesthetics.
@verdverm Looks good, thank you! Merged it
@verdverm Published as spinjs@0.4.161
@verdverm Looks good, but lets add docs as well for the new options into README.md