-
Notifications
You must be signed in to change notification settings - Fork 96
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 support for Cache-Control header #821
Conversation
+1 a5a4560 |
The |
Add support for Cache-Control header Reviewed-by: kuenishi
r_t passed. |
@borshop merge |
@junamai2000 Thank you for contribution. Though I already merged, by any chance could you write tests? It's easy, put some codes with boto into client_test. Or riak_test would be also fine. |
I wrote test codes for this change, but something wrong in my environment and I can not run the test codes. Could you run tests for me if I make a pull request? |
Sure! In some cases running Python client_tests could be difficult due to version dependency or whatever. |
Could you check my codes? #836 |
@junamai2000 Thank you for contribution. We'll merge it. |
Supporting Cache-Control header is useful to communicate with CDN. S3 supports this header.