Skip to content
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

Add access to json body content via req and beresp #44

Closed
malud opened this issue Oct 7, 2020 · 0 comments · Fixed by #34
Closed

Add access to json body content via req and beresp #44

malud opened this issue Oct 7, 2020 · 0 comments · Fixed by #34
Assignees
Labels
enhancement New feature or request theme/config

Comments

@malud
Copy link
Collaborator

malud commented Oct 7, 2020

Accessing a bodies json payload is crucial for at least header enrichment in both directions. This will be a useful option for some new features later on.

The request (req) and response (beresp) evaluation needs to be extended with json_body.
Since we have to buffer and may rewind the body content for further processing a limit is a requirement:

request/response_body_limit or body_limit for both directions.

Buffering those bodies should be known in general on startup/config load. Detecting the json_body usage within a backend block. This enables some kind of lazy loading for json_body if used in config and with the http method matches POST PUT or PATCH and content-type application/json.

@malud malud added the enhancement New feature or request label Oct 7, 2020
@malud malud self-assigned this Oct 7, 2020
@malud malud linked a pull request Oct 7, 2020 that will close this issue
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 9, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 19, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 20, 2020
…iven (json_body) #44

Does not affect post form parsing

Introduce test helper
malud pushed a commit that referenced this issue Oct 20, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
…iven (json_body) #44

Does not affect post form parsing

Introduce test helper
malud pushed a commit that referenced this issue Oct 23, 2020
malud pushed a commit that referenced this issue Oct 23, 2020
…ing as fallback #44

Add test for json 'null' values
malud pushed a commit that referenced this issue Oct 23, 2020
…ing as fallback #44

Add test for json 'null' values
malud pushed a commit that referenced this issue Oct 26, 2020
…ing as fallback #44

Add test for json 'null' values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request theme/config
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant