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.

Sign up for GitHub

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

Jump to bottom

[2.7] bpo-34791: xml package obeys ignore env flags (GH-9544) #9547

Merged
merged 1 commit into from Oct 19, 2018
Merged

[2.7] bpo-34791: xml package obeys ignore env flags (GH-9544) #9547

merged 1 commit into from Oct 19, 2018

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Sep 24, 2018 •
edited by bedevere-bot

The xml.sax and xml.dom.domreg modules now obey
sys.flags.ignore_environment.

Signed-off-by: Christian Heimes christian@python.org

https://bugs.python.org/issue34791
(cherry picked from commit 223e501)

Co-authored-by: Christian Heimes christian@python.org

https://bugs.python.org/issue34791

@tiran @miss-islington
bpo-34791: xml package obeys ignore env flags (pythonGH-9544)
b1401ec 
The xml.sax and xml.dom.domreg modules now obey
sys.flags.ignore_environment.

Signed-off-by: Christian Heimes <christian@python.org>

https://bugs.python.org/issue34791
(cherry picked from commit 223e501)

Co-authored-by: Christian Heimes <christian@python.org>
@bedevere-bot bedevere-bot mentioned this pull request Sep 24, 2018
Merged
@bedevere-bot bedevere-bot added the type-security A security issue label Sep 24, 2018
@miss-islington
Copy link
Contributor Author

@tiran: Status check is done, and it's a success ✅ .

@vstinner vstinner merged commit 2546ac8 into python:2.7 Oct 19, 2018
@miss-islington miss-islington deleted the backport-223e501-2.7 branch October 19, 2018 23:09
kianoosh64
kianoosh64 reviewed Nov 21, 2018
View reviewed changes
Copy link

@kianoosh64 kianoosh64 left a comment

Choose a reason for hiding this comment

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

requested change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kianoosh64 kianoosh64 kianoosh64 left review comments

Assignees
No one assigned
Labels
type-security A security issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@miss-islington @kianoosh64 @vstinner @tiran @the-knights-who-say-ni @bedevere-bot