-
Notifications
You must be signed in to change notification settings - Fork 8k
gh-attestation cmd integration #8698
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
Merged
Merged
Changes from all commits
Commits
Show all changes
143 commits
Select commit
Hold shift + click to select a range
4337447
add attestation cmd
malancas fa39995
add codeowners
malancas 2be3de6
update args passed to the attestation cmd
malancas cd221b7
rename file
malancas e3ae656
use gh-attestation branch for passing iostreams from the root
malancas 24865a6
add package security team entry to codeowners
malancas 2ccc34a
Merge branch 'trunk' into gh-attestation-cmd
malancas a082815
start moving over verify cmd and general verification code
malancas 3a08c03
clean up common and verify specific policy code
malancas 97c10f4
move artifact package over
malancas 5462df5
start pulling in the github api client wrapper
malancas 7f3e818
fix imports
malancas 82bd13b
add logger and test packages
malancas 501add4
add additional packages to support verify command
malancas ff1eb37
fix mock api client
malancas fe5d85e
clean up mock api client
malancas a340e9b
include missing fields
malancas b8a570f
use correct owner
malancas b1fbfdd
add more mock api client options
malancas cbd57de
add download cmd
malancas 884fe22
add inspect cmd
malancas f832d57
pass factory object to inspect cmd, add inspect sub cmd to attestatio…
malancas e35bec9
add verify-tuf-root cmd
malancas fa22214
pass iostream struct from command
malancas 4df1548
rename logger pkg to logger
malancas 31a00b4
fix path in codeowners
malancas 149a78a
formatter
malancas 7fddc8f
Merge branch 'trunk' into gh-attestation-cmd
malancas 532fe95
go mod tidy
malancas 9c3d1e5
Merge branch 'gh-attestation-cmd' of github.com:malancas/cli into gh-…
malancas fec5bfb
fix printf linter issue
malancas 625c441
fix printf linter issue
malancas af90403
check user's GH host for compatibility
malancas 6507122
pass oci client to commands directly
malancas 56261d7
rename command
malancas 2923d10
mark tuf-root-verify cmd hidden
malancas 155a7c9
move client initialization back to subcommands
malancas 4273980
add more verbose options and logging
malancas 277af34
add missing logger
malancas 5492f89
add testing around OCI and API client
malancas 956e10a
add integration test
malancas 3e34139
Merge branch 'trunk' into gh-attestation-cmd
malancas e756694
fix file path
malancas 1a7fa8e
Merge branch 'gh-attestation-cmd' of github.com:malancas/cli into gh-…
malancas 9b9ddcd
fix command
malancas 617a9f3
build executable before integration test
malancas b973914
split integration tests
malancas e171f97
remove integration test steps
malancas e43d98f
fix flag value
malancas b64e003
run integration tests on ubuntu for now
malancas fbba7b5
pull over doc updates
malancas 5ad63c0
delete unused test data
malancas 5dbdb3e
Merge branch 'trunk' into gh-attestation-cmd
malancas b42fc73
Merge branch 'trunk' into gh-attestation-cmd
malancas cf54f05
remove Go patch version
malancas e0fc8c7
switch assert to require
malancas 71480af
rename file
malancas 8f412af
move integration tests to prexisting test workflow
malancas fbb27df
use platform matrix for integration tests
malancas ac79032
simplify build step
malancas f1a2f8f
use StringEnumFlag handling
malancas b8c9f96
typo
malancas be6b042
use the iostreams.Test helper func
malancas f055517
create interface for oci client
malancas 38adc43
add tests for oci client
malancas 54db553
rename files
malancas 4690166
format file
malancas cd1c30a
fix shellcheck issues
malancas f78e104
use testing TempDir method
malancas 63b5872
cleanup unused tempdir handling
malancas 3ee14bb
use table driven tests
malancas 658fab2
check correct cmd
malancas 0018903
support repo option in download sub cmd
malancas 9ad3b22
switch over to using RunE
malancas 4ae0470
unexport top level subcommand funcs
malancas 310a54f
add comment around keychain option
malancas 7565cc4
update comments
malancas be8ce2b
fix inconsistent naming
malancas 9becdc5
add tests for CLI commands
malancas af7f699
Merge branch 'trunk' into gh-attestation-cmd
malancas 01260ef
check for noattestationsfound err
malancas 06801fb
try out metadata abstraction instead
malancas 39a76fd
switch to using MetadataStore abstraction
malancas 7f477af
include test case with failing metadata store
malancas 8e3c197
look for err specific to file write
malancas 3c2ded1
unexport fields
malancas 2cf02a4
return err when an unsupported hash alg is provided
malancas 587b318
PrintTableToStdOut returns err when rendering fails
malancas 5787fef
start adding sigstore verifier unit tests
malancas 99111a3
add more sigstore verifier specific tests
malancas 11eba5e
use cli table printer
malancas 338f9cf
return JSON results in slice instead of table
malancas ff1e8e4
move mock client to test file
malancas 24412d2
remove unneeded table printer method
malancas cf8f0ea
add initial tests for tufrootverify cmd
malancas 8177c6c
formatting
malancas 79fa437
cleanup method
malancas ead84a2
close file in error handling branch
malancas 1ba04c6
normalize artifact path
malancas 67ac04b
remove unneeded embedded file system
malancas 7067130
include image name reference err
malancas 007d368
use GH_DEBUG value for io handling
malancas af75fe8
remove quiet and verbose flags
malancas 2ef8eb6
add more tufrootveriify tests
malancas b5f6bd5
GitHubTUFOptions no longer needs to return error
malancas 08f0a56
remove unneeded slice
malancas 274af8b
normalize all relative paths
malancas 5cc2f6a
clean up nil client checks
malancas cbcd34e
set api server based on host
malancas b37bf1f
Merge branch 'trunk' into gh-attestation-cmd
malancas 53df7cd
add comment about http client
malancas 172e4f6
use format flag to handle json output in verify cmd
malancas 09b11db
use format flag to handle json output
malancas 6abd1fe
Merge branch 'trunk' into gh-attestation-cmd
malancas 4f2f719
use normalized path for cli test arg
malancas b5ffcf1
Merge branch 'gh-attestation-cmd' of github.com:malancas/cli into gh-…
malancas 12c7de2
add tests for json output
malancas 02a1274
cleanup error wrapping
malancas 807eefd
use test fixtures correctly by normalizing path
malancas 1e51ec7
Merge branch 'trunk' into gh-attestation-cmd
malancas 2a00a02
dont clean
malancas 5172edd
escape backwards slash for windows files with replace
malancas 023b2af
Merge branch 'trunk' into gh-attestation-cmd
malancas 79d3c95
use strings.Split func
malancas 98d7bef
use strings.Replace for all command tests
malancas 3391fb1
Merge branch 'trunk' into gh-attestation-cmd
malancas 49dfa3f
use CLI cache dir to store tuf metadata
malancas c16406b
Merge branch 'gh-attestation-cmd' of github.com:malancas/cli into gh-…
malancas 2e1d27a
Tweaked docstrings for gh attestation download
phillmv 416d981
Tweaked docstrings for gh attestation verify
phillmv e9e1b09
Fix for bug in gh attestation where the wrong hostname was being pass…
phillmv 96da5f7
lets hide tuf-root-verify eh?
phillmv 0b14695
Forgot verify's short str.
phillmv 6b4f387
Merge branch 'trunk' into gh-attestation-cmd
malancas de63dea
Merge pull request #2 from phillmv/gh-attestation-cmd
malancas 64c3fd0
Merge pull request #1 from phillmv/fix-verify-api-requests
malancas c0ceb99
add remote verification test
malancas bb3b6ab
Revert "add remote verification test"
malancas 2522b10
update json result handling
malancas ad0b36c
add json tags to struct returned by command
malancas 308a2c4
fix how json results are handled
malancas ccd1cc4
add test to ensure JSON output is valid
malancas ded4bea
Merge branch 'trunk' into gh-attestation-cmd
malancas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.