Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Resolve minor security warnings #3140

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
Merged
merged 9 commits into from
Jul 18, 2025
Merged

Resolve minor security warnings #3140

merged 9 commits into from
Jul 18, 2025

Conversation

J12934
Copy link
Member

@J12934 J12934 commented Jul 18, 2025

Description

Resolves some minor warnings from github & sonarqube which should hopefully mark the ci as green again.

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

J12934 added 5 commits July 18, 2025 10:31
@J12934
Disable script execution during npm installs
e979e90 
Signed-off-by: Jannik Hollenbach <[email protected]>
@J12934
Disable service account automounting for nuclei cache jobs
1552c0d 
They don't need kubernetes api access

Signed-off-by: Jannik Hollenbach <[email protected]>
@J12934
Remove unused vars
63bfc66 
Signed-off-by: Jannik Hollenbach <[email protected]>
@J12934
Resolve code smell warning
16e3750 
Signed-off-by: Jannik Hollenbach <[email protected]>
@J12934
Minor package updates
a39a6c8 
Signed-off-by: Jannik Hollenbach <[email protected]>
@J12934 J12934 self-assigned this Jul 18, 2025
@netlify Netlify
Copy link

netlify bot commented Jul 18, 2025 •
edited
Loading

✅ Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit a054157
🔍 Latest deploy log https://app.netlify.com/projects/docs-securecodebox/deploys/687a17a156b1290008d265d8

@J12934 J12934 moved this from Triage to To Review in secureCodeBox Jul 18, 2025
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@github-project-automation github-project-automation bot moved this from To Review to Reviewer Approved in secureCodeBox Jul 18, 2025
@Reet00 Reet00 merged commit 147ee8e into secureCodeBox:main Jul 18, 2025
49 checks passed
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in secureCodeBox Jul 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Reet00 Reet00 Reet00 approved these changes

Assignees

@J12934 J12934

Labels
maintenance
Projects
secureCodeBox
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@J12934 @Reet00