-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Clarify CookieContainer thread-safety #11171
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
Clarify CookieContainer thread-safety #11171
Conversation
Tagging subscribers to this area: @dotnet/ncl |
Learn Build status updates of commit b3f9a00:
|
File | Status | Preview URL | Details |
---|---|---|---|
xml/System.Net/CookieContainer.xml | View | Details |
xml/System.Net/CookieContainer.xml
- Line 0, Column 0: [Warning: disallowed-html-tag - See documentation]
HTML tag 'format' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
Co-authored-by: Genevieve Warren <[email protected]>
Learn Build status updates of commit 26d016e:
|
File | Status | Preview URL | Details |
---|---|---|---|
xml/System.Net/CookieContainer.xml | View | Details |
xml/System.Net/CookieContainer.xml
- Line 0, Column 0: [Warning: disallowed-html-tag - See documentation]
HTML tag 'format' isn't allowed. Replace it with approved Markdown or escape the brackets if the content is a placeholder.
For more details, please refer to the build report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:
- Try searching the learn.microsoft.com contributor guides
- Post your question in the Learn support channel
Learn Build status updates of commit 5d2c10c: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit d6fac13: ✅ Validation status: passed
For more details, please refer to the build report. For any questions, please:
|
This PR supplements dotnet/docs#45658 by clarifying that
CookieContainer
methods are thread-safe but also noting the dangers of unanticipadedCookieContainer
sharing.Fixes #1535
Fixes #10770