-
Notifications
You must be signed in to change notification settings - Fork 109
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
[argus] few tweaks #4925
[argus] few tweaks #4925
Conversation
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.
LGTM
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.
Just a minor merge conflict fix needed otherwise I think this is good to go.
bce9585
to
74fc629
Compare
Good call @mnaamani, changed to a single header with comma-separated values. Also rebased and fixed conflicts |
Access-Control-Expose-Headers
response header to allow web clients checking cache status of downloaded file.http
logs