Permalink
Browse files

Re-enable SSL bumping during Sauce Labs testing

SSL bumping interferes with WebSocket traffic. This is why the feature
was originally disabled.

Running WPT locally depends on the use of self-signed certificates. SSL
bumping avoids the errors that self-signed certificates would otherwise
trigger in the browser under test.

Due to this situation, it is not currently possible to correctly test
both WebSocket behavior and HTTPS behavior. The latter should be
preferred because it is fundamental to the web platform.
  • Loading branch information...
jugglinmike committed Dec 4, 2018
1 parent a1adf1e commit 4754901eab9b5d709d0366ab8c4c03999ddf23f1
Showing with 1 addition and 13 deletions.
  1. +1 −13 provisioning/configuration/roles/sauce-labs/files/sc-wrapper
@@ -4,18 +4,6 @@
# the Sauce Connect binary. This script serves as an intermediary which
# enables some options and passes any additional options along to the
# "true" Sauce Connect binary.
#
# > Sauce Connect executes a type of "man-in-the-middle" interception of
# > encrypted test traffic through SSL bumping, and decrypts that traffic in
# > the course of the test. [...] The disadvantage to SSL bumping is that some
# > network components, such as browsers and servers that use WebSockets, won't
# > work if the traffic to them has been altered, which Sauce Connect appears
# > to do. [...] You can disable SSL bumping by using the `-B` /
# > `--no-ssl-bump-domains` argument when you start Sauce Connect and
# > specifying which domains should not be bumped or specify all so that all
# > domains that passed through the tunnel are not bumped .
#
# Source: https://wiki.saucelabs.com/display/DOCS/Troubleshooting+Sauce+Connect#TroubleshootingSauceConnect-DisablingSSLBumping

import signal
import subprocess
@@ -27,7 +15,7 @@ signal.signal(signal.SIGINT, lambda _, __: proc.kill())
signal.signal(signal.SIGTERM, lambda _, __: proc.terminate())

args = [
'--no-ssl-bump-domains=all', '--logfile', '/var/log/sauce-connect/sc.log'
'--logfile', '/var/log/sauce-connect/sc.log'
] + sys.argv[1:]

proc = subprocess.Popen(['sc'] + args)

0 comments on commit 4754901

Please sign in to comment.