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.

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

fix: dir package isolates system/user directory #161

Closed

Conversation

JeyJeyGao
Copy link
Contributor

config add EnvelopeType argument.
dir package optimize: allow user define DirLevel = UserLevel | SystemLevel | UnionLevel.
refector config pacakge: receive a path argument when load and save file to allow user to overwrite the default directory structure.
Test: added unit test.

Resolves #144
Signed-off-by: Junjie Gao junjiegao@microsoft.com

JeyJeyGao and others added 17 commits September 9, 2022 11:37
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Binbin Li <libinbin@microsoft.com>
Follow up change from
notaryproject#147 (comment)

Signed-off-by: Pritesh Bandi <pritesb@amazon.com>
…otaryproject#131)

## What
Refactor `notation-go` to support multiple envelope types.
Background can be checked in
notaryproject/notation#278
I wthe whole PR into two PRs to help review, this is the first PR. More
unit test cases will be added in the next PR.
The whole picture is here
notaryproject#146

## Major Changes
- Use package `github.com/notaryproject/notation-core-go/signature` to
sign and verify.
- Combine `runner` and `signer` into a `provider` for `pluginSigner` to
sign and remove the `pluginSigProvider`.
- Add `builtinProvider` to support local signing and `externalProvider`
to support signing by plugin.
- Move the payload media type and its checks to `signature` package as
mentioned in notaryproject/notation-core-go#73
- Support new
[keySpec](https://github.com/notaryproject/notaryproject/blob/main/signature-specification.md#algorithm-selection)
and plugin contract.
- Get verification plugin and version from extended attributes.
- Add `SpeculateSignatureEnvelopeFormat` to inspect signature (This
function may change later to better inspect a signature)
- Add sign/verify from file test cases.
Signed-off-by: zaihaoyin <zaihaoyin@microsoft.com>

Signed-off-by: zaihaoyin <zaihaoyin@microsoft.com>
Co-authored-by: zaihaoyin <zaihaoyin@microsoft.com>
Signed-off-by: zaihaoyin <zaihaoyin@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
receive dirLevel for function in PathManager

Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
Signed-off-by: Junjie Gao <junjiegao@microsoft.com>
@shizhMSFT shizhMSFT added the dev Issue or PR related to dev branches label Oct 11, 2022
@JeyJeyGao JeyJeyGao closed this Oct 17, 2022
@JeyJeyGao JeyJeyGao deleted the feat/dir_add_writeLevel branch November 10, 2022 01:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev Issue or PR related to dev branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants