-
Notifications
You must be signed in to change notification settings - Fork 302
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
Fix: set Content-type to JSON #2599
base: master
Are you sure you want to change the base?
Conversation
Thanks for this @arminfelder! |
6108f11
to
443e108
Compare
Signed-off-by: Armin Felder <armin.felder@gmail.com>
443e108
to
2295e23
Compare
@jhaals I did a reformat and rebase, should be fine now |
Thanks for the rebase. I was just about to merge this but I realized that the OPTIONS call fails for the preview build. It's probably the case that the /secret endpoint need to support OPTIONS as well https://github.com/jhaals/yopass/blob/master/pkg/server/server.go#L150 |
adds config option for Access-Control-Allow-Origin
@jhaals I just added implementations for OPTIONS method calls + a config option to override the default '*' |
@jhaals as 11.18 is now released, deploy/netlify check passes, so we could go forward with this one |
Sets the content-type to application/json
solves issues with modsecurity
Fixes #2600