Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[docs] Describe bypassing the proxy #2354

Merged
merged 8 commits into from
May 10, 2018

Conversation

VasilyStrelyaev
Copy link
Collaborator

\cc @DevExpress/testcafe-docs @AlexKamaev

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit 08a9934 have failed. See details.

@VasilyStrelyaev
Copy link
Collaborator Author

@testcafe-build-bot \retest

@testcafe-build-bot
Copy link
Collaborator

❌ Tests for the commit 08a9934 have failed. See details.

Copy link
Collaborator

@kirovboris kirovboris left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/r-


The `rules` parameter takes a comma-separated list (without spaces) of URLs that require direct access. You can omit parts of the URLs or replace them with wildcards `*` (which has the same effect). TestCafe will correspond these symbols to any number of characters in the URL.

```sh
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd describe shortly what these examples mean.

@@ -361,7 +368,11 @@ runner.useProxy('proxy.corp.mycompany.com');
```

```js
runner.useProxy('172.0.10.10:8080');
runner.useProxy('172.0.10.10:8080', ['localhost:8080', 'internal-resource.corp.mycompany.com']);
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 9697eba have passed. See details.

testcafe chrome my-tests/**/*.js --proxy proxy.corp.mycompany.com --proxy-bypass localhost:8080,internal-resource.corp.mycompany.com
```

The `*.mycompany.com` value means that all URLs in `mycompany.com` subdomains will be accessed directly.
Copy link
Contributor

@AlexKamaev AlexKamaev Apr 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we mention that the rule *.mycompany.com is the same as .mycompany.com (without *)?

I mean that it is not required to add an asterisk in the beginning or in the end of rule, but it's required in the middle of rule (.mycompany.*.com)

Copy link
Collaborator Author

@VasilyStrelyaev VasilyStrelyaev Apr 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I didn't think about wildcards in the middle

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit a1d5da2 have passed. See details.

@kirovboris
Copy link
Collaborator

ping @DevExpress/testcafe-docs

@VasilyStrelyaev
Copy link
Collaborator Author

@dirk-pieterse , please check the docs

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 44ba52a have passed. See details.

@AndreyBelym AndreyBelym added this to the Sprint #11 milestone May 8, 2018
@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit af0837b have passed. See details.

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 1c923db have passed. See details.

@testcafe-build-bot
Copy link
Collaborator

✅ Tests for the commit 994a5e9 have passed. See details.

@MargaritaLoseva MargaritaLoseva merged commit 8e85a4d into DevExpress:master May 10, 2018
@VasilyStrelyaev VasilyStrelyaev deleted the bypass-proxy branch May 17, 2018 14:39
kirovboris pushed a commit to kirovboris/testcafe-phoenix that referenced this pull request Dec 18, 2019
* Describe bypassing the proxy

* Address Boris' remarks

* Address Alexey's remarks

* A grammar correction

* Update command-line-interface.md

* Update runner.md

* Update command-line-interface.md

minor changes

* Update runner.md

minor changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants