-
Notifications
You must be signed in to change notification settings - Fork 653
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
add =
character to tag validation regex
#2008
add =
character to tag validation regex
#2008
Conversation
= is only supported in EC2 not in S3 tags, they don't document it properly. The regex we have is well tested against AWS S3. Please provide more evidence, test it with S3 and then let us reopen this PR if needed. |
As per the doc linked in the package:
In our organization we compute md5 hashes (which can contain |
@harshavardhana I confirm : I just successfully manually added a tag containing |
Please provide a code sample |
When relaxing minio-go restrictions, using mc:
|
can you try |
|
The tags restrictions mentioned for the
validTagKeyValue
regex includes the=
character which is not present in the regex.This PR aims at reducing this drift between AWS and Minio restrictions on this API.
I've added a simple test case in the unit test for this package, let me know if this enough or if further testing is needed.