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.

Sign up for GitHub

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

Jump to bottom

fix: allow any content using starts-with with an empty value #1718

Merged

harshavardhana merged 1 commit into minio:master from fortuneforme:post-policy-allow-any-content-using-starts-with

Oct 31, 2022

Contributor

fortuneforme commented Oct 26, 2022

Thanks for the PR @skirsten

commit 4acfae712723fae025df6b841f9cc0e80c8ca464
Author: Simon Kirsten <[email protected]>
Date:   Fri Oct 21 22:38:04 2022 +0200

    Allow empty contentTypeStartsWith in PostPolicy (#1710)

But it's still not allowed to add an empty value in addNewPolicy, so i supported it when using starts-with.
And by the way, this PR will allow empty keyStartsWith in PostPolicy.


          fix: allow any content using starts-with with an empty value

minio-trusted approved these changes

View reviewed changes

harshavardhana approved these changes

View reviewed changes

vadmeste approved these changes

View reviewed changes

Member

vadmeste left a comment

LGTM

harshavardhana merged commit d6fbd49 into minio:master

fortuneforme deleted the post-policy-allow-any-content-using-starts-with branch

November 1, 2022 01:13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet