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

Response Modifications Toxic #1

Merged
merged 19 commits into from
Jul 15, 2022
Merged

Response Modifications Toxic #1

merged 19 commits into from
Jul 15, 2022

Conversation

avaakash
Copy link
Member

@avaakash avaakash commented Jul 8, 2022

This PR adds the following toxics

  1. Response Status Code Modification
    toxic-name: status_code
    attributes: status_code, modify_response_body

  2. Response Body Modification
    toxic-name: modify_body
    attributes: body

  3. Response Header Modification
    toxic-name: header
    attributes: headers, mode

avaakash added 5 commits June 28, 2022 02:32
@avaakash
Added http status code toxic
32c7555 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Added test for status code toxic
f44737e 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Added modify body toxic; Added 2xx body template
c2dd0d8 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Added build script; Improved toxics
ddb23c8 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Added push workflow
cc5470d 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash avaakash self-assigned this Jul 8, 2022
github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 8, 2022
View reviewed changes
toxics/header.go Fixed Show fixed Hide fixed
toxics/header.go Fixed Show fixed Hide fixed
avaakash added 14 commits July 11, 2022 12:16
@avaakash
removed debug code
613a41e 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Improved httputils code to be more concise
8c7bd27 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
sanitising user input for logs
797b19a 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Changes to fix the golangci-lint errors
17aa800 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
yaml-lint fixes
a6fb7c6 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
reverted build command and added prod-build
7fbbed1 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
yaml-lint fixes
9abbbcd 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
fixed header test case
3b145e2 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
merged with master
ef3ada1 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
minor fix
13751db 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
update go version
09175a5 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
removed go<1.17 from test
cd7f79a 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
added unused package check
cf60d68 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
reverted branch addition
3cce633 
Signed-off-by: Akash Shrivastava <akash.shrivastava@harness.io>
@avaakash
Copy link
Member Author

Issue for Test CI failing: #2
Will be fixed in subsequent PR.

@avaakash avaakash merged commit e01437c into master Jul 15, 2022
@avaakash avaakash deleted the http-status-code branch March 1, 2023 16:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jonsy13 Jonsy13 Jonsy13 approved these changes

@uditgaurav uditgaurav uditgaurav approved these changes

@neelanjan00 neelanjan00 Awaiting requested review from neelanjan00

Assignees

@avaakash avaakash

Labels
Ready-to-Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@avaakash @uditgaurav @Jonsy13