After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 334805 - AccountDialog URL validation
AccountDialog URL validation
Status: RESOLVED FIXED
Product: f-spot
Classification: Other
Component: General
CVS
Other All
: Normal critical
: ---
Assigned To: F-spot maintainers
F-spot maintainers
: 334804 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-03-16 19:53 UTC by Ivan Zlatev
Modified: 2006-03-16 20:29 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
proposed patch (414 bytes, patch)
2006-03-16 19:54 UTC, Ivan Zlatev
none Details | Review

Description Ivan Zlatev 2006-03-16 19:53:11 UTC
Steps to reproduce:
1. Add new gallery with url ftp://gnome.org
2. Click on OK


Stack trace:
System.NotSupportedException: ftp://i-nz.net/gallery_remote2.php
in <0x0017a> System.Net.WebRequest:GetCreator (System.String prefix)
in <0x0001c> System.Net.WebRequest:Create (System.Uri requestUri)
in <0x00077> FormClient:Submit (System.Uri uri, FSpot.ProgressItem 
progress_item)
in <0x0000f> FormClient:Submit (System.Uri uri)
in <0x000de> GalleryRemote.Gallery:DetectGalleryVersion (System.String url)
in <0x0004b> FSpot.GalleryAccount:.ctor (System.String name, System.String url, 
System.String username, System.String password, GalleryVersion version)
in <0x003a6> FSpot.GalleryAccountManager:ParseAccount (System.Xml.XmlNode node)
in <0x000e3> FSpot.GalleryAccountManager:ReadAccounts ()


Other information:
Patch included
Comment 1 Ivan Zlatev 2006-03-16 19:54:05 UTC
Created attachment 61389 [details] [review]
proposed patch

My patch to fix this.
Comment 2 Larry Ewing 2006-03-16 20:12:55 UTC
*** Bug 334804 has been marked as a duplicate of this bug. ***
Comment 3 Larry Ewing 2006-03-16 20:29:04 UTC
ok I committed a slightly different version of this patch.  Thanks for the patch and for catching this.