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.

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

Sec-fixes #598

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Sec-fixes #598

wants to merge 17 commits into from

Conversation

ATGardner
Copy link
Collaborator

@ATGardner ATGardner commented Oct 4, 2024 •
edited
Loading

  • update golang to 1.23
  • upgraded github.com/argoproj/argo-cd/v2 v2.10.0 => v2.13.0
  • upgraded github.com/go-jose/go-jose/v4 v4.0.2 => v4.0.4
  • upgraded github.com/briandowns/spinner v1.23.0 => v1.23.1
  • replaced github.com/ghodss/yaml with sigs.k8s.io/yaml
  • upgraded github.com/spf13/viper v1.18.2 => v1.19.0

@ATGardner ATGardner self-assigned this Oct 4, 2024
This was referenced Oct 4, 2024
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@ATGardner
update golang to 1.23
7f7d6d4 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
upgraded github.com/argoproj/argo-cd/v2 v2.10.0 => v2.13.0
1c4bd80 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
upgraded github.com/go-jose/go-jose/v4 v4.0.2 => v4.0.4
ac7630a 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
upgraded github.com/briandowns/spinner v1.23.0 => v1.23.1
05e6448 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
replaced github.com/ghodss/yaml with sigs.k8s.io/yaml
a32d08b 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
upgraded github.com/spf13/viper v1.18.2 => v1.19.0
4b615d7 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
updated golangci-lint to v1.61.0
c4301f4 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
fixed lint
f131018 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
fixed tests
cc3fb71 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
ran codegen
ef0f5f2 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner
fixed typo in description
5d9f869 
Signed-off-by: Noam Gal <[email protected]>
@ATGardner ATGardner mentioned this pull request Oct 4, 2024
Closed
@ATGardner
fix codegen sec vulns
a1c531f 
Signed-off-by: Noam Gal <[email protected]>
This was referenced Oct 4, 2024
Closed
Closed
@codecov-commenter
Copy link

codecov-commenter commented Oct 4, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 40.66%. Comparing base (98519df) to head (7e9aa3f).

Files with missing lines Patch % Lines
cmd/commands/project.go 0.00% 1 Missing ⚠️
pkg/kube/kube.go 0.00% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #598      +/-   ##
==========================================
- Coverage   40.67%   40.66%   -0.01%     
==========================================
  Files          41       41              
  Lines        6732     6730       -2     
==========================================
- Hits         2738     2737       -1     
+ Misses       3880     3879       -1     
  Partials      114      114

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ATGardner
Copy link
Collaborator Author

/ci

1 similar comment
@ATGardner
Copy link
Collaborator Author

/ci

@ATGardner
Copy link
Collaborator Author

/ci

@ATGardner ATGardner added the security needed for prisma check label Oct 7, 2024
@ATGardner
Copy link
Collaborator Author

/ci

2 similar comments
@ATGardner
Copy link
Collaborator Author

/ci

@ATGardner
Copy link
Collaborator Author

/ci

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ATGardner ATGardner

Labels
security needed for prisma check
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ATGardner @codecov-commenter