Change variable names: req, ctx, bereq, beresp #181
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
request, context, backend_requests, backend_responses
filex
reviewed
Apr 6, 2021
docs/README.md
Outdated
|
|
||
| #### `bereq` (modified backend request) variable | ||
| `backend_requests.<label>` ist a list of all backend requests, and their variables. |
There was a problem hiding this comment.
Suggested change
| `backend_requests.<label>` ist a list of all backend requests, and their variables. | |
| `backend_requests.<label>` is a list of all backend requests and their variables. |
docs/README.md
Outdated
|
|
||
| #### `bereq` (modified backend request) variable | ||
| `backend_requests.<label>` ist a list of all backend requests, and their variables. | ||
| To access a specific request use the related label. Request and Proxy blocks without a label will be |
There was a problem hiding this comment.
Suggested change
| To access a specific request use the related label. Request and Proxy blocks without a label will be | |
| To access a specific request use the related label. `request` and `proxy` blocks without a label will be |
docs/README.md
Outdated
|
|
||
| #### `beresp` (original backend response) variable | ||
| `backend_responses.<label>` ist a list of all backend responses, and their variables. Same behaviour as for `backend_requests`. |
There was a problem hiding this comment.
Suggested change
| `backend_responses.<label>` ist a list of all backend responses, and their variables. Same behaviour as for `backend_requests`. | |
| `backend_responses.<label>` is a list of all backend responses and their variables. Same behaviour as for `backend_requests`. |
docs/README.md
Outdated
| #### `beresp` (original backend response) variable | ||
| `backend_responses.<label>` ist a list of all backend responses, and their variables. Same behaviour as for `backend_requests`. | ||
| Use the related label to access a specific response. | ||
| Request and Proxy blocks without a label will be available as `default`. |
There was a problem hiding this comment.
Suggested change
| Request and Proxy blocks without a label will be available as `default`. | |
| `request` and `proxy` blocks without a label will be available as `default`. |
docs/README.md
Outdated
| @@ -709,13 +713,13 @@ definitions { | |||
| ### Path Parameter | |||
|
|
|||
| An endpoint label could be defined as `endpoint "/app/{section}/{project}/view" { ... }` | |||
| to access the named path parameter `section` and `project` via `req.path_param.*`. | |||
| to access the named path parameter `section` and `project` via `request.path_param.*`. | |||
There was a problem hiding this comment.
Suggested change
| to access the named path parameter `section` and `project` via `request.path_param.*`. | |
| to access the named path parameter `section` and `project` via `request.path_params.*`. |
alex-schneider
approved these changes
Apr 7, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
request, context, backend_requests, backend_responses
Closes #180