8000 Update Various Go Package by J12934 · Pull Request #1176 · secureCodeBox/secureCodeBox · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@J12934
Copy link
Member
@J12934 J12934 commented May 16, 2022

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 J12934 added the dependencies Pull requests that update a dependency file label May 16, 2022
@J12934 J12934 self-assigned this May 16, 2022
@github-actions
Copy link
github-actions bot commented May 16, 2022

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 J12934 merged commit c47f554 into main May 17, 2022
@J12934 J12934 deleted the fix/go-package-updates branch May 17, 2022 11:25
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.

3 participants

0