Skip to content

Commit

Permalink
OPA object support (#8383)
Browse files Browse the repository at this point in the history
* add api for exposing auth decision data to envoy

* add changelog

* Adding changelog file to new location

* Deleting changelog file from old location

* Adding changelog file to new location

* Deleting changelog file from old location

* Adding changelog file to new location

* Deleting changelog file from old location

* Adding changelog file to new location

* Deleting changelog file from old location

* Adding changelog file to new location

* Deleting changelog file from old location

* Adding changelog file to new location

* Deleting changelog file from old location

* Adding changelog file to new location

* Deleting changelog file from old location

* update after code review

* Adding changelog file to new location

* Deleting changelog file from old location

* api changes

---------

Co-authored-by: changelog-bot <changelog-bot>
Co-authored-by: soloio-bulldozer[bot] <48420018+soloio-bulldozer[bot]@users.noreply.github.com>
  • Loading branch information
tjons and soloio-bulldozer[bot] authored Jul 13, 2023
1 parent 889ff8a commit d6241c8
Show file tree
Hide file tree
Showing 8 changed files with 788 additions and 742 deletions.
4 changes: 4 additions & 0 deletions changelog/v1.15.0-beta20/add-ext-auth-decision-api.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
changelog:
- type: NON_USER_FACING
description: >-
Add field to OPA API to eventually support ext-auth returning objects from OPA decisions.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -651,6 +651,8 @@ spec:
properties:
fastInputConversion:
type: boolean
returnDecisionReason:
type: boolean
type: object
query:
type: string
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1078,6 +1078,14 @@ message OpaAuthOptions {
// are included in the request input. All other fields are dropped. Dropped fields will not be evaluated by the OPA engine.
// By default, this is set to false and all fields are evaluated by OPA.
bool fast_input_conversion = 1;

// Return the reason given from the OPA engine after a decision made on this policy. Reason must be the second
// parameter of the query and will be a protobuf struct if the reason is an object, and otherwise will be a json
// string. The entry will be in the returned DynamicMetadata in the CheckResponse and the structure will be
// envoy.filters.http.ext_authz:
// -> name of the auth step, i.e. spec.configs[i].name
// -> reason
bool return_decision_reason = 2;
}

// Authenticates and authorizes requests by querying an LDAP server. Gloo makes the following assumptions:
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1,503 changes: 761 additions & 742 deletions projects/gloo/pkg/api/v1/enterprise/options/extauth/v1/extauth.pb.go

Large diffs are not rendered by default.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit d6241c8

Please sign in to comment.