Skip to content

netrw: patch 9.2.0073 should take into account a possible port number#19533

Closed
MiguelBarro wants to merge 1 commit intovim:masterfrom
MiguelBarro:netrw/hostname
Closed

netrw: patch 9.2.0073 should take into account a possible port number#19533
MiguelBarro wants to merge 1 commit intovim:masterfrom
MiguelBarro:netrw/hostname

Conversation

@MiguelBarro
Copy link
Contributor

echomsg 'NetRead Usage:'
echomsg ':Nread machine:path uses rcp'
echomsg ':Nread "machine path" uses ftp with <.netrc>'
echomsg ':Nread "machine id password path" uses ftp'
echomsg ':Nread dav://machine[:port]/path uses cadaver'
echomsg ':Nread fetch://machine/path uses fetch'
echomsg ':Nread ftp://[user@]machine[:port]/path uses ftp autodetects <.netrc>'
echomsg ':Nread http://[user@]machine/path uses http wget'
echomsg ':Nread file:///path uses elinks'
echomsg ':Nread https://[user@]machine/path uses http wget'
echomsg ':Nread rcp://[user@]machine/path uses rcp'
echomsg ':Nread rsync://machine[:port]/path uses rsync'
echomsg ':Nread scp://[user@]machine[[:#]port]/path uses scp'
echomsg ':Nread sftp://[user@]machine[[:#]port]/path uses sftp'

The port number may appear at the end of the hostname in several protocols: cadaver, ftp, rsync, scp & sftp.

@MiguelBarro MiguelBarro marked this pull request as ready for review February 28, 2026 17:23
Signed-off-by: Guybrush <miguel.barro@live.com>
@chrisbra
Copy link
Member

chrisbra commented Mar 1, 2026

thanks

@chrisbra chrisbra closed this in a619852 Mar 1, 2026
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Mar 2, 2026
…e validation

Problem:  netrw: does not take port into account in hostname validation
          (after v9.2.0073)
Solution: Update hostname validation check and test for an optional port
          number (Miguel Barro)

closes: vim/vim#19533

vim/vim@a619852

Co-authored-by: Miguel Barro <miguel.barro@live.com>
zeertzjq added a commit to zeertzjq/neovim that referenced this pull request Mar 2, 2026
…e validation

Problem:  netrw: does not take port into account in hostname validation
          (after v9.2.0073)
Solution: Update hostname validation check and test for an optional port
          number (Miguel Barro)

closes: vim/vim#19533

vim/vim@a619852

Co-authored-by: Miguel Barro <miguel.barro@live.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants