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

Add allocator option: IgnoreCertErrors #809

Closed
wants to merge 2 commits into from

Conversation

shelld3v
Copy link
Contributor

@shelld3v shelld3v commented May 2, 2021

No description provided.

@ZekeLu
Copy link
Member

ZekeLu commented May 2, 2021

Thanks for the contribution!

There are lots of command line options for the Chrome browser (a non-official and a little outdated list). I'm not sure whether this one is special enough to get a name. And if we want to give it a name, I think IgnoreCertErrors looks more accurate.

@ZekeLu ZekeLu changed the title Add command line option: IgnoreCert Add allocator option: IgnoreCertErrors May 4, 2021
@ZekeLu
Copy link
Member

ZekeLu commented Jun 26, 2021

It turns out that this option is useful (I used it several times when using a proxy to capture the network traffics), and the cited web page does not include this option, so I'm going to merge this PR.

Thank you!

@ZekeLu ZekeLu closed this Jun 26, 2021
@shelld3v
Copy link
Contributor Author

@ZekeLu: It turns out that this option is useful, so I'm going to merge this PR
Also @ZekeLu: closed #809

@ZekeLu
Copy link
Member

ZekeLu commented Jun 27, 2021

Sorry for making you confuse. I want to move this option after ProxyServer and modify the comment, so I can't merge this PR directly. This PR is closed after the cherrypick. You can find the contribution in the commit history. Thank you!

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.

None yet

2 participants