Skip to content

Commit

Permalink
docs: Promote user-info-fetcher docs to navgation (#557)
Browse files Browse the repository at this point in the history
* docs: Promote user-info-fetcher docs to navgation

* Document future plans

* Apply suggestions from code review

Co-authored-by: Techassi <[email protected]>

---------

Co-authored-by: Techassi <[email protected]>
  • Loading branch information
sbernauer and Techassi authored Jul 23, 2024
1 parent 8b6beac commit c9684b9
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
10 changes: 10 additions & 0 deletions docs/modules/opa/pages/usage-guide/user-info-fetcher.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,16 @@ NOTE: The OAuth2 Client in Keycloak must be given the `view-users` _Service Acco
// TODO: Document how to use it in OPA regorules, e.g. to authorize based on group membership
== Example rego rule

[NOTE]
.User-facing API & API stability
====
Since the 24.07 SDP release we provide an example rego rule in our documentation using an HTTP request.
However, our plan is that the user-facing API of the SPD is *not* the HTTP API of user-info-fetcher, but instead regorules that will automatically be shipped to the OPA server.
This enables us to make underlying (for example breaking) changes to the HTTP API while keeping the rego rules API stable.
The documentation will be updated to use the deployed rego rules once available.
====

[NOTE]
.About unencrypted HTTP
====
Expand Down
2 changes: 1 addition & 1 deletion docs/modules/opa/partials/nav.adoc
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
* xref:opa:usage-guide/index.adoc[]
** xref:opa:usage-guide/listenerclass.adoc[]
** xref:opa:usage-guide/policies.adoc[]
// ** xref:opa:usage-guide/user-info-fetcher.adoc[] Not yet ready for public consumption
** xref:opa:usage-guide/user-info-fetcher.adoc[]
** xref:opa:usage-guide/resources.adoc[]
** xref:opa:usage-guide/logging.adoc[]
** xref:opa:usage-guide/monitoring.adoc[]
Expand Down

0 comments on commit c9684b9

Please sign in to comment.