Feature #1514
Parsing ftp URL
Status: | Closed | Start date: | 2008-06-22 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | - | % Done: | 0% | |
Category: | Wiki | |||
Target version: | 0.8 | |||
Resolution: | Fixed |
Description
Patch Redmine 0.7.1 for Parsing ftp URLs
Index: lib/redmine/wiki_formatting.rb =================================================================== --- lib/redmine/wiki_formatting.rb (revision 1496) +++ lib/redmine/wiki_formatting.rb (working copy) @@ -126,6 +126,7 @@ ) ( (?:https?://)| # protocol spec, or + (?:ftp://)| # (?:www\.) # www.* ) (
History
#1
Updated by Jean-Philippe Lang over 14 years ago
- Category set to Wiki
- Status changed from New to Closed
- Target version set to 0.8
- Resolution set to Fixed
Applied in r1577.
#2
Updated by Kevin Light over 14 years ago
- Status changed from Closed to Reopened
I think the regex for parsing ftp:// links is a bit too aggressive as it tears apart sftp:// links.
The less aggressive approach would be to watch for whitespace before the ftp:// so the patch should be:
Index: lib/redmine/wiki_formatting.rb =================================================================== --- lib/redmine/wiki_formatting.rb (revision 1496) +++ lib/redmine/wiki_formatting.rb (working copy) @@ -126,6 +126,7 @@ ) ( (?:https?://)| # protocol spec, or + (?:\sftp://)| # (?:www\.) # www.* ) (
#3
Updated by Stanislav German-Evtushenko over 14 years ago
Kevin Light wrote:
I think the regex for parsing ftp:// links is a bit too aggressive as it tears apart sftp:// links.
The less aggressive approach would be to watch for whitespace before the ftp:// so the patch should be:
[...]
You can use "inline code" to disable parsing. For example: sftp://myserv
#4
Updated by Jean-Philippe Lang about 14 years ago
- Status changed from Reopened to Closed
sftp and ftps proto are now properly turned into links (r2018).