GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |
GeezerVison is showing github.com — Document explicit certificate stores for restricted CA trust by OskarEichler · Pull Request #61 · lostisland/faraday-net_http Conversation
Contributor
SummaryDocument the existing additive CA behavior and show how to restrict trust using an explicit certificate store. Related to #14; this deliberately preserves the existing compatibility policy. Reproduction and verificationSeven bounded localhost TLS exchanges use two generated synthetic CAs. Default trust and a matching CA succeed; ca_file/ca_path alone retain additive behavior. Adding an explicit empty store correctly rejects the unrelated CA, as does an explicit unrelated cert_store. Only temporary public certificate files are written; private keys remain in memory. SSL_CERT_FILE/SSL_CERT_DIR changes are confined to the reproduction process and restored. All loopback servers and threads close. A candidate change to the default policy was evaluated locally but is excluded from this PR and the consumer branch after reviewing #14's compatibility discussion.
LimitationsNo test/spec files added or modified: the originating repository forbids them. Focused reproductions ran outside the repository. Other Ruby versions and platforms have not been run locally. The PR contains only this focused change. Breaking-change notesNone. Documentation only; certificate/hostname verification and default trust behavior are unchanged.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. Resolve README conflict by keeping both the SOCKS proxy and restricted certificate authority sections. Co-authored-by: Copilot <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ReviewersNo reviewsUh oh!
There was an error while loading. Please reload this page. AssigneesNo one assignedLabels
None yet
ProjectsMilestoneNo milestoneDevelopmentSuccessfully merging this pull request may close these issues. Uh oh!
There was an error while loading. Please reload this page. Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later. | |
| You are visitor 01402520 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |