-
Notifications
You must be signed in to change notification settings - Fork 116
feat: Support delegated targets roles in repo writer #175
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
ethan-lowman-dd
merged 48 commits into
theupdateframework:master
from
mnm678:delegations
Apr 19, 2022
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
302d2fc
Support delegated targets roles in repo writer
ethan-lowman-dd ab3d2f3
Add back return err in writeTopLevelTargetWithExpires
ethan-lowman-dd b3ea3ab
Remove space
ethan-lowman-dd a235746
Move hash bin impl to pkg/targets
ethan-lowman-dd 0facfb0
Fix build after rebase
ethan-lowman-dd 08b738b
Make AddTargetsWithDigest support delegated targets
ethan-lowman-dd 9991710
Remove unused GetPublicKeys
ethan-lowman-dd 619e3b2
Revert "Remove unused GetPublicKeys"
ethan-lowman-dd 087db51
Fix typo
ethan-lowman-dd c6cabeb
Easier hash bin usage when setting up delegations
ethan-lowman-dd 50f2540
Remove comment that ChangePassphrase doesn't support delegated roles
ethan-lowman-dd 0854893
Update comments
ethan-lowman-dd 759410f
Add API to reset delegations
ethan-lowman-dd 57656df
clean up verifySignatures
ethan-lowman-dd 5a0932d
Fix tests for ChangePassphrase
ethan-lowman-dd 9e694fc
Fix comments
ethan-lowman-dd 4c94b6f
Deduplicate key IDs when setting up hash bins
ethan-lowman-dd f5f045d
Revert "Revert "Remove unused GetPublicKeys""
ethan-lowman-dd da82948
Add warning about GetSigners. TODO: fix usage
ethan-lowman-dd 2b81fbe
Clean up some usage of setMetawithSigners
ethan-lowman-dd 601a815
Remove setMetaWithSigners since it's easy to misuse
ethan-lowman-dd 36b49f9
cleanup
ethan-lowman-dd a44b8e9
cleanup
ethan-lowman-dd b403fc0
Cleanup usage of delegatorDBs
ethan-lowman-dd 167f279
Simplify AddTargetsWithExpiresToPreferredRole
ethan-lowman-dd 19cb8a4
Remove debugging
ethan-lowman-dd b95b1e5
Remove targetsMetaWithSigners
ethan-lowman-dd 414a9c3
Fix comment
ethan-lowman-dd 2c35b27
Turn FIXME into an issue
ethan-lowman-dd 1b228ef
Fix comment position
ethan-lowman-dd eebf224
Remove out-of-place comment
ethan-lowman-dd c2f8cc6
Add missing hyphen
ethan-lowman-dd 21f9d6c
Test delegations construction more rigorously
ethan-lowman-dd 54b4224
Support removing delegated targets files
ethan-lowman-dd b28aafe
Fix tests
ethan-lowman-dd b512eec
Add more tests for delegation
ethan-lowman-dd bd0f486
Fix unchecked error
ethan-lowman-dd 733c142
Clarify comment
ethan-lowman-dd 56093d4
Better naming
ethan-lowman-dd b63ab01
Support delegated targets in Sign and AddOrUpdateSignature
ethan-lowman-dd e14e3e7
Add test for ResetTargetsDelegationsWithExpires
ethan-lowman-dd ecd63d1
Improve naming of delegation methods
ethan-lowman-dd 17c7318
Update tests for int64 version numbers
ethan-lowman-dd 501b1f5
Add tests for Sign and AddOrUpdateSignature with delegations
ethan-lowman-dd 805310c
Remove unused functions in errors.go
ethan-lowman-dd 6ebf1e8
Remove commented out test code
ethan-lowman-dd 722aa3e
Add test to verify that adding a delegation to an existing role doesn…
ethan-lowman-dd e481cd3
Fix SignMessage usage in test with new canonicalization pattern
ethan-lowman-dd 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.