[Webkit-unassigned] [Bug 83259] [BlackBerry] Networking - Assertion failed when it store FTP credentials

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=83259





--- Comment #4 from Sean Wang <xuewen.wang at torchmobile.com.cn>  2012-04-11 03:33:53 PST ---
(In reply to comment #3)
> (From update of attachment 135975 [details])
> I think that the correct answer is to not use HTTP credential storage code for ftp. It's specifically tailored for HTTP credential semantics.

Currently, only the CF and BlackBerry porting are using the CredentialStorage as I grepped CredentialStorage::set. There are 3 type browsers which support FTP credential storage. They are Firefox 11.0, Chromium 18.0 and IE 8 as I tested. So I think storing the FTP credentials makes sense.

As the credentials are stored in class ProtectionSpace, and it supports many protocols in its ProtectionSpaceServerType and it supports ServerType comparing in its "operator ==",  So storing other protocol's credentials should work with the current implementations.

What is your concern?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.