Closed Bug 166396 Opened 21 years ago Closed 21 years ago

embedders should be able to override the default socket type used for HTTP connections


(Core :: Networking: HTTP, defect, P2)






(Reporter: darin.moz, Assigned: darin.moz)


(Keywords: arch, topembed, Whiteboard: [need-branch-approval])


(1 file, 2 obsolete files)

embedders should be able to override the default socket type used for HTTP
connections.  we should provide a preference for this.
Priority: -- → P2
Target Milestone: --- → mozilla1.2alpha
Attached patch v1 patch (obsolete) — Splinter Review
Comment on attachment 97611 [details] [diff] [review]
v1 patch

Do you have any fallback plans?  For example, suppose that a pref pointing to a
socket transport provider is left but the provider is removed from disk.

Also, what about overriding the default http SSL socket?
Attachment #97611 - Flags: review+
good point... this could definitely could be expanded upon, and a sanity check
that a nsISocketProvider exists corresponding to the preference value is
probably needed.  ok, i'll rev the patch.  as for SSL, i think we should prefer
our builtin code for that.  i don't think we want to support an external SSL
socket implementation just yet.  anyways, it could always use XPCOM to make
itself the nsISocketProvider implementation of ssl.  this pref is about using a
socket type when none would normally be used.
Attached patch v2 patch (obsolete) — Splinter Review
same patch, but now we try to instantiate the specified socket provider to
verify that the value provided in the preferences actually works.  if not, then
we fallback on the default setting (which is NULL).
Attachment #97611 - Attachment is obsolete: true
Comment on attachment 97708 [details] [diff] [review]
v2 patch

what about the ssl override?

     if (mConnectionInfo->UsingSSL())

also, what if the pref is set but it is empty.	You should do something like:
+	 if (NS_SUCCEEDED(rv) && val)
can preferences really return NS_OK with a NULL result?  despite that, your
comment made me think of another problem... if someone wanted to reset the
preference to its default value, they'd want to call SetCharPref(..., "") with
an empty string value.  but, the pref observer code would fail to modify
mDefaultSocketType on account of the fact that "" is not a valid socket type. 
oops!  new patch coming up ;-)
Attached patch v3 patchSplinter Review
Attachment #97708 - Attachment is obsolete: true
Comment on attachment 97785 [details] [diff] [review]
v3 patch

got an r=dougt over AIM.
Attachment #97785 - Flags: review+
Comment on attachment 97785 [details] [diff] [review]
v3 patch

a=chofmann for the trunk
Attachment #97785 - Flags: approval+
fixed-on-trunk (for mozilla 1.2 alpha)
Closed: 21 years ago
Keywords: mozilla1.0.2
Resolution: --- → FIXED
Whiteboard: [need-branch-approval] for 1.0 branch.  Change mozilla1.0.2+ to fixed1.0.2 when
checked in 
verified - checked with lxr that patch was applied
Please verify the bug. Once verified, change the keyword fixed1.0.2 to
Tom, can you do the same verification within LXR for the mozilla 1.0 branch?
yes, verified patch applied to 1.0 branch
You need to log in before you can comment on or make changes to this bug.