Closed Bug 761529 Opened 8 years ago Closed 8 years ago

ssltunnel broken in non-proxy mode

Categories

(Testing :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla16

People

(Reporter: mayhemer, Assigned: mayhemer)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Attached patch v1Splinter Review
Reporting in xpcshell since this touches it.

Bug 572570 broke ssltunnel non-proxy mode by preventing an ssl update on the client socket.  So, ssl trafic is forwarded to the target server (httpd.js).

This is a simple patch that fixes that, while preserving websocket tests (@content/base/tests) functional.

Next time please let ssltunnel changes pass also through me, thanks.
Attachment #630108 - Flags: review?(jgriffin)
Comment on attachment 630108 [details] [diff] [review]
v1

Review of attachment 630108 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks, and sorry for the bustage.
Attachment #630108 - Flags: review?(jgriffin) → review+
This is a try build, for ref: https://tbpl.mozilla.org/?tree=Try&rev=b96a4d3b0b9c
https://hg.mozilla.org/mozilla-central/rev/c93e04192c57

(Merged by Ed Morley)
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
Component: XPCShell Harness → ssltunnel
QA Contact: xpcshellharness → ssltunnel
Component: ssltunnel → General
You need to log in before you can comment on or make changes to this bug.