-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Checksum Trivy download #32
Labels
correction
This is incorrect and needs altering
Comments
drew-viles
added a commit
that referenced
this issue
Apr 11, 2023
drew-viles
added a commit
that referenced
this issue
Apr 12, 2023
Previously there were calls to viper.GetSomething all over the code. This was prone to error if a value changed so now the code has been refactored to process any viper inputs. Flags will always override the viper config file and any fields not supplied will use the defaults where applicable. Also added checksum validation for the Trivy cli download Fixes #32 & Fixes #33
drew-viles
added a commit
that referenced
this issue
Apr 12, 2023
Previously there were calls to viper.GetSomething all over the code. This was prone to error if a value changed so now the code has been refactored to process any viper inputs. Flags will always override the viper config file and any fields not supplied will use the defaults where applicable. Added checksum validation for the Trivy cli download Removed duplicate Trivy report structs and generally moved some bits around to begin work for allowing other clouds to be used. There is still work to be done to allow other clouds though as it's still generally OpenStack focussed. Fixes #32 & Fixes #33
drew-viles
added a commit
that referenced
this issue
Apr 12, 2023
Previously there were calls to viper.GetSomething all over the code. This was prone to error if a value changed so now the code has been refactored to process any viper inputs. Flags will always override the viper config file and any fields not supplied will use the defaults where applicable. Added checksum validation for the Trivy cli download Removed duplicate Trivy report structs and generally moved some bits around to begin work for allowing other clouds to be used. There is still work to be done to allow other clouds though as it's still generally OpenStack focussed. Added new docs/readme & changelog Fixes #32 & Fixes #34 updated README
drew-viles
added a commit
that referenced
this issue
Apr 12, 2023
Previously there were calls to viper.GetSomething all over the code. This was prone to error if a value changed so now the code has been refactored to process any viper inputs. Flags will always override the viper config file and any fields not supplied will use the defaults where applicable. Added checksum validation for the Trivy cli download Removed duplicate Trivy report structs and generally moved some bits around to begin work for allowing other clouds to be used. There is still work to be done to allow other clouds though as it's still generally OpenStack focussed. Added new docs/readme & changelog Fixes #32 & Fixes #34
drew-viles
added a commit
that referenced
this issue
Apr 12, 2023
Previously there were calls to viper.GetSomething all over the code. This was prone to error if a value changed so now the code has been refactored to process any viper inputs. Flags will always override the viper config file and any fields not supplied will use the defaults where applicable. Added checksum validation for the Trivy cli download Removed duplicate Trivy report structs and generally moved some bits around to begin work for allowing other clouds to be used. There is still work to be done to allow other clouds though as it's still generally OpenStack focussed. Added new docs/readme & changelog Fixes #32 & Fixes #34
drew-viles
added a commit
that referenced
this issue
Apr 12, 2023
Previously there were calls to viper.GetSomething all over the code. This was prone to error if a value changed so now the code has been refactored to process any viper inputs. Flags will always override the viper config file and any fields not supplied will use the defaults where applicable. Added checksum validation for the Trivy cli download Removed duplicate Trivy report structs and generally moved some bits around to begin work for allowing other clouds to be used. There is still work to be done to allow other clouds though as it's still generally OpenStack focussed. Added new docs/readme & changelog Fixes #32 & Fixes #34
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I might be tempted to check the checksum of this file against what Aqua publish and verify its authenticity here...
Originally posted by @yankcrime in #31 (comment)
The text was updated successfully, but these errors were encountered: