-
-
Notifications
You must be signed in to change notification settings - Fork 371
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
Support multiple mutation handlers: update documentation #193
Conversation
@@ -2027,8 +2027,13 @@ | |||
"match": { | |||
"$ref": "#/definitions/ruleMatch" | |||
}, | |||
"mutator": { | |||
"$ref": "#/definitions/ruleHandler" | |||
"mutators": { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be updated automatically by the CI :) You can keep it as is - it will simply be overwritten once we merge the oathkeeper PR
@@ -657,9 +657,9 @@ Status Code **200** | |||
| »» match | [ruleMatch](#schemarulematch) | false | none | none | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For future reference - this will also be auto-generated from swagger spec :) See warning at the top:
You are viewing a REST API documentation. This documentation is auto-generated from a swagger specification which itself is generated from annotations in the source files of the project. It is possible that this documentation includes bugs and that code samples are incomplete or wrong.
If you find issues in the respective documentation, please do not edit the markdown files directly (as they are generated) but raise an issue on the project's GitHub instead. This documentation will improve over time with your help! If you have ideas how to improve this part of the documentation, feel free to share them in a GitHub issue any time.
You can leave this as is however - it will be automatically updated once we merge oathkeeper pr
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, my bad. I was mindlessly hunting those mutators 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
all good :)
update documentation to reflect changes implemented in ory/oathkeeper#233