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
downloader: log message on unexpected bundle content type #4432
downloader: log message on unexpected bundle content type #4432
Conversation
Not sure if debug is the right log level, but since a wrong content-type header could be used to serve a valid bundle, it would likely be noisy to log this at higher levels. |
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.
Some nitpicks and questions but this look OK otherwise 👍
d057df9
to
bdcb239
Compare
175316b
to
c6c0593
Compare
I feel stuck on the race condition. Any help would be appreciated, @srenatus 🙂 |
a6698f7 maybe something along those lines? |
fc3a14a
to
b86e61a
Compare
Fixes open-policy-agent#4278 Signed-off-by: Anders Eknert <anders@eknert.com>
b86e61a
to
01d82bc
Compare
Fixes #4278
Signed-off-by: Anders Eknert anders@eknert.com