FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update Various Go Package by J12934 · Pull Request #1176 · secureCodeBox/secureCodeBox · GitHub

Update Various Go Package - #1176

Merged
J12934 merged 3 commits into
mainfrom
fix/go-package-updates
May 17, 2022
Merged

Update Various Go Package#1176
J12934 merged 3 commits into
mainfrom
fix/go-package-updates

Conversation

J12934 commented May 16, 2022

Copy link
Copy Markdown
Member

Description

Updates golang packages for the operator and auto-discovery.
Resolves most usages of vulnerable version of protobuf lib: GHSA-c3h9-896r-86jm

Vulnerable version is still included transitivly via controller-runtime -> prometheus but these should not acutally be used and we might not be able to upgrade controller-runtime without a major / breaking release. See https://github.com/kubernetes-sigs/controller-runtime/releases/tag/v0.12.0

With the updates I've also updated the kubernetes client libs. These introduced new fields to our crds as the new kubernetes version has new features. These new fields will also cause all ScheduledScans created with RetriggerOnScanTypeChange=true to be triggered, as the hash gets recalculated for the new fields

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure npm test runs for the whole project.

J12934 added 3 commits May 16, 2022 17:26
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
Hash is calculated for the entire object, so adding new fields to it changes the hash

This will cause all scans created with `RetriggerOnScanTypeChange` to be recreated as the hash gets recalculated for the new fields

Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
J12934 added the dependencies Pull requests that update a dependency file label May 16, 2022
J12934 self-assigned this May 16, 2022

github-actions Bot commented May 16, 2022
edited
Loading

Copy link
Copy Markdown

MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ DOCKERFILE dockerfilelint 1 0 0.37s
✅ DOCKERFILE hadolint 1 0 0.07s
✅ GIT git_diff yes no 0.08s
⚠️ GO golangci-lint 1 1 3.14s
✅ GO revive 1 0 0.01s
✅ SPELL misspell 8 0 0.2s
⚠️ YAML prettier 6 1 4.64s
✅ YAML v8r 6 0 4.15s
⚠️ YAML yamllint 6 1 7.91s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

J12934 merged commit c47f554 into main May 17, 2022
J12934 deleted the fix/go-package-updates branch May 17, 2022 11:25
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL