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

Backport #50740 to 2017.7 #50873

Merged
merged 3 commits into from Dec 14, 2018

Conversation

Projects
None yet
3 participants
@Ch3LL
Copy link
Contributor

commented Dec 14, 2018

Backport #50740 to 2017.7

mtorromeo and others added some commits Dec 4, 2018

Catch UnicodeDecodeErrors in id verification
Do not crash if the id cannot be decoded. An invalid id provided by an untrusted client can DOS the master.

@garethgreenaway garethgreenaway self-requested a review Dec 14, 2018

@garethgreenaway garethgreenaway merged commit e7f0ab6 into saltstack:2017.7 Dec 14, 2018

3 of 10 checks passed

jenkins/pr/docs Testing docs...
Details
jenkins/pr/py2-centos-7 running py2-centos-7...
Details
jenkins/pr/py2-ubuntu-1604 running py2-ubuntu-1604...
Details
jenkins/pr/py2-windows-2016 running py2-windows-2016...
Details
jenkins/pr/py3-centos-7 running py3-centos-7...
Details
jenkins/pr/py3-ubuntu-1604 running py3-ubuntu-1604...
Details
jenkins/pr/py3-windows-2016 running py3-windows-2016...
Details
WIP Ready for review
Details
continuous-integration/jenkins/pr-merge This commit looks good
Details
jenkins/pr/lint Python lint test has passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.