SauceTunnel: "direct-domains" are not comma-separated

[vladikoff]

Ref: https://docs.saucelabs.com/reference/sauce-connect/#command-line-options

Currently the SauceTunnel does not comma separate, but space separates the domains:

[csnover]

The documentation you link, as well as sc --help, specify that the argument is to be comma-separated, which is what Dig Dug does. “Comma-separated list of domains.” This has been true back to Sauce Connect 3. If Sauce Connect 4 is not accepting a comma-separated list, then it is a bug in Sauce Connect 4 that needs to be fixed there.

[vladikoff]

You can see from my screenshot that part of the command is

-D latest.dev.lcip.org [SPACE] restmail.net

shouldn't that be:

-D latest.dev.lcip.org,restmail.net

Maybe I'm misunderstanding something

[csnover]

Dig Dug does comma-separate the list of direct domains:

this.directDomains.length && args.push('-D', this.directDomains.join(','));

This report sounded to me like Sauce Connect expected a space-separated list, which is what my response was trying to address, since the code already generates a comma-separated list.

Without a reduced set of reproduction steps, expected results, and actual results, it is very difficult to understand what is actually being reported, so I will guess blindly and incorrectly much of the time in such cases. To minimise confusion, please make sure to try to provide them for all issues in future. Thanks!

[vladikoff]

Dig Dug does comma-separate the list of direct domains:

Yeah saw that in the source, but I wonder why the resulting process has it space separated.