-
Notifications
You must be signed in to change notification settings - Fork 512
Add inspection v2 (always encrypted) option. #2386
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
Conversation
Keep v1 if non-encrypted.
# Conflicts: # go.sum
+ clean up file name.
|
Still pending mc 4185, but short of |
# Conflicts: # go.mod # go.sum
|
@klauspost PTAL at the tests |
|
@harshavardhana It is a cyclic dependency. Minio depends on this and this depends on minio |
Then can we skip it? @klauspost as a non-failure? |
bexsoft
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
@bexsoft Yes, the encryption option requires the minio PR to be running to work (and the PR above requires this console version). |
Depends on: minio/minio#15474 for functionality (which depends on this) Blocks: minio/minio#15474 Keep v1 if non-encrypted and decrypt it. Otherwise if encrypted, use a fixed public key.

Depends on: minio/minio#15474 for functionality (which depends on this)
Blocks: minio/minio#15474
Keep v1 if non-encrypted and decrypt it.
Otherwise if encrypted, use a fixed public key.
Will not work until minio is upgraded with as well.