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.

Sign up for GitHub

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

Jump to bottom

add options for replacing or patching the body #12

Merged
merged 2 commits into from
Aug 9, 2021
Merged

add options for replacing or patching the body #12

merged 2 commits into from
Aug 9, 2021

Conversation

Hexilee
Copy link
Member

@Hexilee Hexilee commented Aug 2, 2021 •
edited
Loading

Signed-off-by: hexilee i@hexilee.me

  • add update_content_length option for replacing or patching the body, true by default; fix chaos-mesh#2152
  • change the config structure of the body replacing contents, support TEXT and BASE64 types.

@Hexilee
add update_content_length option for patching or replacing body; add …
563c050 
…TEXT option for replacing body

Signed-off-by: hexilee <i@hexilee.me>
@Hexilee
update examples
f3a86c5 
Signed-off-by: hexilee <i@hexilee.me>
@Hexilee
Copy link
Member Author

Hexilee commented Aug 2, 2021

PTAL @STRRL @Andrewmatilde

@STRRL
Copy link
Member

STRRL commented Aug 3, 2021

change the config structure of the body replacing contents, support TEXT and JSON types.

TEXT and BASE64?

@Hexilee
Copy link
Member Author

Hexilee commented Aug 3, 2021

change the config structure of the body replacing contents, support TEXT and JSON types.

TEXT and BASE64?

Fixed

@Hexilee Hexilee merged commit b2208fb into master Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@STRRL STRRL STRRL approved these changes

@Andrewmatilde Andrewmatilde Awaiting requested review from Andrewmatilde

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

HTTPChaos patch response not worked as expected, HTTP Header content-length is not updated
2 participants
@Hexilee @STRRL