MantisBT - NetSurf
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0000526NetSurf[All Projects] Generalpublic2012-05-10 23:472017-09-09 12:06
ReporterChris Young 
Assigned ToMichael Drake 
PrioritynormalSeverityminorReproducibilityalways
StatusassignedResolutionopen 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Fixed in CI build #
Reported in CI build #
URL of problem page
Summary0000526: URIs with embedded auth details containing '@'
DescriptionNetSurf fails to interpret the following URI correctly:
http://guest@chrisy:guest1@webdav.mydrive.ch/public/test.txt

Instead of visiting webdav.mydrive.ch with a username of guest@chrisy, it tries to visit host chrisy.

URL-encoding the first @ makes NetSurf interpret the address as expected. ie. the following works where the above fails:
http://guest%40chrisy:guest1@webdav.mydrive.ch/public/test.txt
Additional InformationImported from sourceforge bug http://sourceforge.net/support/tracker.php?aid=3525663 on Tue Dec 10 17:34:12 2013

TagsNo tags attached.
Relationships
Attached Files

Notes
(0000070)
Vincent Sanders   
2013-12-18 17:31   
I reproduced this with todays master branch
(0000135)
Vincent Sanders   
2013-12-24 16:35   
NetSurf needs to deal with the failure case of parsing a uri better

http://url.spec.whatwg.org/#parsing
(0001013)
Dave Higton   
2015-11-01 11:51   
Confirmd with CI#3043, RISC OS build.
(0001586)
Daniel Silverstone   
2017-09-09 12:06   
The nsurl parser still has this issue, Michael needs to take a look.

Issue History
Date ModifiedUsernameFieldChange
2013-12-10 17:41Vincent SandersNew Issue
2013-12-10 17:41Vincent SandersStatusnew => assigned
2013-12-10 17:41Vincent SandersAssigned To => Sourceforge Import placeholder
2013-12-18 17:31Vincent SandersNote Added: 0000070
2013-12-18 17:31Vincent SandersAssigned ToSourceforge Import placeholder =>
2013-12-18 17:31Vincent SandersReproducibilityhave not tried => always
2013-12-18 17:31Vincent SandersStatusassigned => confirmed
2013-12-18 17:31Vincent SandersResolutionno change required => open
2013-12-18 17:31Vincent SandersDescription Updatedbug_revision_view_page.php?rev_id=431#r431
2013-12-18 17:31Vincent SandersAdditional Information Updatedbug_revision_view_page.php?rev_id=433#r433
2013-12-24 16:35Vincent SandersNote Added: 0000135
2015-11-01 11:51Dave HigtonNote Added: 0001013
2017-09-09 12:06Daniel SilverstoneAssigned To => Michael Drake
2017-09-09 12:06Daniel SilverstoneStatusconfirmed => assigned
2017-09-09 12:06Daniel SilverstoneNote Added: 0001586