Remove explicit "Authorization" header #155

Merged

RNCTX merged 2 commits from master into master

2020-07-14 19:22:04 +02:00

RNCTX commented

2020-07-14 18:01:13 +02:00

(Migrated from github.com)

Previously included explicit declaration of Authorization header based on #32 . Tested with Oauth2 plugin against a Postgrest API with Postman as a client, and the explicit Authorization header/variable are not necessary on Nginx 1.18.0, running on Ubuntu 20.04. I was able to POST and PATCH with the returned token just fine without explicit declaration of the Authorization header in the Nginx config.

Previously included explicit declaration of `Authorization` header based on #32 . Tested with Oauth2 plugin against a Postgrest API with Postman as a client, and the explicit `Authorization` header/variable are not necessary on Nginx 1.18.0, running on Ubuntu 20.04. I was able to POST and PATCH with the returned token just fine without explicit declaration of the `Authorization` header in the Nginx config.

babelouest commented

2020-07-14 18:18:33 +02:00

(Migrated from github.com)

Thanks for the upgrade @RNCTX !

There's a conflict in your pull request, due to desynchronized INSTALL.md files between your fork and mine, can you resolve the conflict so I can merge the PR?

Thanks for the upgrade @RNCTX ! There's a conflict in your pull request, due to desynchronized INSTALL.md files between your fork and mine, can you resolve the conflict so I can merge the PR?

👍 1