Skip to content

Novatechflow/dependabot vuln remediation#136

Merged
novatechflow merged 4 commits intoKafScale:mainfrom
novatechflow:novatechflow/dependabot-vuln-remediation
Mar 22, 2026
Merged

Novatechflow/dependabot vuln remediation#136
novatechflow merged 4 commits intoKafScale:mainfrom
novatechflow:novatechflow/dependabot-vuln-remediation

Conversation

@novatechflow
Copy link
Copy Markdown
Collaborator

Summary

  • Remediates fixable Dependabot vulnerabilities in Go and npm dependencies.
  • Addresses multiple open Code Scanning findings (CodeQL + Scorecard) in code, workflows, and Dockerfiles.
  • Keeps @aws-sdk/xml-builder pinned while forcing safe transitive fast-xml-parser.

Testing

  • go test ./...
  • make test-produce-consume (broker changes)
  • make test-consumer-group (group changes)

Checklist

  • Added/updated unit tests for new logic
  • Added/updated e2e coverage for bug fixes
  • Added license headers to new files

@novatechflow
Copy link
Copy Markdown
Collaborator Author

@kamir / @klaudworks - pls review

@klaudworks
Copy link
Copy Markdown
Collaborator

The go tests for nested modules such as addons/processors/sql-processor are currently not part of the CI pipelines.
I noticed these tests failing cause of a missing go.sum entry for golang.org/x/crypto/bcrypt.

 go test ./internal/metadata

go mod tidy should fix it.

@novatechflow
Copy link
Copy Markdown
Collaborator Author

The go tests for nested modules such as addons/processors/sql-processor are currently not part of the CI pipelines. I noticed these tests failing cause of a missing go.sum entry for golang.org/x/crypto/bcrypt.

 go test ./internal/metadata

go mod tidy should fix it.

aren't they? let me check.

@novatechflow
Copy link
Copy Markdown
Collaborator Author

good catch @klaudworks! Fixed :)

@novatechflow novatechflow merged commit 06d33bd into KafScale:main Mar 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants