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

Add Jenkins security scan to common files #597

Merged
merged 2 commits into from
Mar 2, 2023

Conversation

NotMyFault
Copy link
Member

New hosting requests are reviewed initially by our set of predefined CodeQL rules. The recent hosting requests subject to the new scanning showed long lists of issues (including a few false-positives).
Therefore, I'm assuming a lack of awareness of how to mitigate, and even better, avoid introducing possible security vulnerabilities in your plugin.

For this reasoning, I would like to include the security scan by default in all new plugins, to raise awareness of possible security issues and give maintainers an opportunity, to react to issues.

Co-authored-by: Tim Jacomb <21194782+timja@users.noreply.github.com>
@NotMyFault NotMyFault merged commit 57f4f5f into jenkinsci:master Mar 2, 2023
@NotMyFault NotMyFault deleted the security-scanning branch March 2, 2023 23:25
@NotMyFault
Copy link
Member Author

@jglick Mind cutting a release, please?

@jglick
Copy link
Member

jglick commented Mar 3, 2023

https://github.com/jenkinsci/archetypes/releases/tag/archetypes-1.18

I somehow botched the commit history a bit but maven-release-plugin 🤷

@NotMyFault
Copy link
Member Author

NotMyFault commented Mar 3, 2023 via email

@@ -0,0 +1,23 @@
# More information about the Jenkins security scan can be found at the developer docs: https://www.jenkins.io/doc/developer/security/scan/
Copy link
Member

Choose a reason for hiding this comment

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

Please reference https://www.jenkins.io/redirect/jenkins-security-scan/ from outside jenkins.io for URL stability.

@jglick jglick mentioned this pull request Mar 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants