feat: for remote_json authorizer, add headers

ory / oathkeeper

A cloud native Identity & Access Proxy / API (IAP) and Access Control Decision API that authenticates, authorizes, and mutates incoming HTTP(s) requests. Inspired by the BeyondCorp / Zero Trust white paper. Written in Go.

https://www.ory.sh/?utm_source=github&utm_medium=banner&utm_campaign=hydra

Apache License 2.0

3.2k stars 349 forks source link

feat: for remote_json authorizer, add headers #1140

Closed divyun closed 4 months ago

divyun commented 7 months ago

✨(authorizers): for remote_json, introduce the ability to specify headers ⬆️ Bump github.com/ory/x to v0.0.605 to pass the build fixing vulnerabilities

Related issue(s)

https://github.com/ory/oathkeeper/issues/969

Checklist

[x] I have read the contributing guidelines.
[x] I have referenced an issue containing the design document if my change introduces a new feature.
[x] I am following the contributing code guidelines.
[x] I have read the security policy.
[x] I confirm that this pull request does not address a security vulnerability. If this pull request addresses a security vulnerability, I confirm that I got the approval (please contact security@ory.sh) from the maintainers to push the changes.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added or changed the documentation.

Further Comments

Corresponding Docs PR https://github.com/ory/docs/pull/1604

CLAassistant commented 7 months ago

All committers have signed the CLA.

codecov[bot] commented 7 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (b5d4d88) 78.16% compared to head (eff951b) 77.58%. Report is 1 commits behind head on master.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #1140 +/- ## ========================================== - Coverage 78.16% 77.58% -0.59% ========================================== Files 80 79 -1 Lines 3898 3992 +94 ========================================== + Hits 3047 3097 +50 - Misses 576 616 +40 - Partials 275 279 +4 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

divyun commented 7 months ago

Code coverage is lowered due to some extra lines being tracked for internal/cloudstorage/setup.go in the code that wasn't changed in the PR. I can only assume that's due to certain dependencies being updated to pass "Code scanning" check.

HEAD: https://app.codecov.io/gh/ory/oathkeeper/pull/1140/tree/internal/cloudstorage BASE: https://app.codecov.io/gh/ory/oathkeeper/commit/b5d4d88e69fc38c0536aef1f00155255683e815d/tree/internal/cloudstorage