Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add official IANA assigned FTPS (FTP over TLS) port #401

Closed
wants to merge 1 commit into from

Conversation

ExE-Boss
Copy link

@ExE-Boss ExE-Boss commented Jul 14, 2018

FTPS (FTP over TLS) was specified in RFC 4217 by IETF, and officially assigned the ports 990 (command/​control) and 989 (data) by IANA soon after.

Because of the TLS everything process that is going on among browser vendors, I feel like adding FTPS to the URL specification is a good addition.


Preview | Diff

  FTPS (FTP over TLS) was specified in RFC 4217 by IETF, and officially
assigned the ports 990 (command/control) and 989 (data) by IANA soon
after.

# External links:
- https://tools.ietf.org/html/rfc4217
@annevk
Copy link
Member

annevk commented Jul 21, 2018

Thanks for contributing this, but I don't think we want to make this change. As far as I know implementations do not parse ftps: URLs the same way as ftp: URLs. They're parsed per the non-special URL rules. Did you find differently?

@ExE-Boss
Copy link
Author

Well, I know that Firefox was working on a FTPS implementation at some point in the past.

@annevk
Copy link
Member

annevk commented Jul 21, 2018

I'm pretty sure that's no longer the case. It's also not entirely clear to me we'd want to change the URL parser for that. It seems it could be supported as a non-special URL just as well.

@annevk
Copy link
Member

annevk commented Aug 21, 2018

Closing this per the above comments. Happy to reconsider once implementers are on board.

@annevk annevk closed this Aug 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants