-
Notifications
You must be signed in to change notification settings - Fork 82
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
Configure Cache-Control and Expires headers on the uploaded files. #53
Configure Cache-Control and Expires headers on the uploaded files. #53
Conversation
@@ -50,7 +50,7 @@ | |||
"core-object": "^1.1.0", | |||
"ember-cli-babel": "^5.0.0", | |||
"ember-cli-deploy-plugin": "^0.2.2", | |||
"lodash": "^3.9.3", | |||
"lodash": "^4.11.1", |
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.
@YoranBrondsema why are we updating this?
I ask because lodash has been giving us quite a bit of gas lately in different plugins and I've seen some weird npm incompatibilities around it
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.
@ghedamat No particular reason but to keep up-to-date. I wasn't aware of any recent problems with lodash, I can revert it back to 3 no problems.
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.
seems stupid but yeah let's revert and maybe do this in another pr
so we can release a beta and get someone to test it just in case.
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.
I understand, no problem.
@YoranBrondsema LGTM just that comment on lodash /cc @lukemelia |
@ghedamat Any news on when this can be merged? We'd like to start using this functionality ourselves. |
done :) thanks! will try to cut a release later today a patch bump should be enough |
@ghedamat Perfect thanks! |
This PR adds the ability to configure the
Cache-Control
andExpires
headers through thecacheControl
andexpires
configuration options.Solves #52