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

tools: Check for unsafe Content Security Policy #7391

Merged
merged 1 commit into from Jul 27, 2017

Conversation

Projects
None yet
2 participants
@stefwalter
Copy link
Contributor

commented Jul 27, 2017

Check for unsafe Content Security Policy in our static code tests.

@stefwalter stefwalter added the blocked label Jul 27, 2017

@stefwalter

This comment has been minimized.

Copy link
Contributor Author

commented Jul 27, 2017

This should be red until the dependent fixes are merged.

@martinpitt

This comment has been minimized.

Copy link
Member

commented Jul 27, 2017

@stefwalter: This is now ready for rebasing.

@martinpitt martinpitt removed the blocked label Jul 27, 2017

# of setenforce 0
#

if grep -E 'content-security-policy.*(\*|unsafe)' pkg/*/*.json*; then

This comment has been minimized.

Copy link
@martinpitt

martinpitt Jul 27, 2017

Member

This new test now needs to bump the total count on the top. I fixed that during rebasing, pushing now.

@martinpitt martinpitt force-pushed the stefwalter:unsafe-static-code branch from 6565b21 to 5463c6c Jul 27, 2017

tools: Check for unsafe Content Security Policy
Check for unsafe Content Security Policy in our static code tests.

Closes #7391

@martinpitt martinpitt merged commit 8c7d5e5 into cockpit-project:master Jul 27, 2017

15 of 16 checks passed

verify/ubuntu-stable 1 tests failed
Details
avocado/fedora-24 Tests passed
Details
container/kubernetes Tests passed
Details
selenium/chrome Tests passed
Details
selenium/firefox Tests passed
Details
semaphoreci The build passed on Semaphore.
Details
verify/centos-7 Tests passed
Details
verify/debian-stable Tests passed
Details
verify/debian-testing Tests passed
Details
verify/fedora-26 Tests passed
Details
verify/fedora-atomic Tests passed
Details
verify/fedora-i386 Tests passed
Details
verify/rhel-7 Tests passed
Details
verify/rhel-7-4 Tests passed
Details
verify/rhel-atomic Tests passed
Details
verify/ubuntu-1604 Tests 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.