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

res.setBody() - Enable custom final responses. #3287

Closed
1 task done
russeree opened this issue Oct 10, 2024 · 1 comment · Fixed by #3299
Closed
1 task done

res.setBody() - Enable custom final responses. #3287

russeree opened this issue Oct 10, 2024 · 1 comment · Fixed by #3299
Labels
bug Something isn't working

Comments

@russeree
Copy link

I have checked the following:

  • I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add

I am doing some preliminary research into the ability to decode and/or modify the response object before writing it to the report.

This pull here seems to add the functionality somewhere, this is not accessible from the GUI on v1.33.0
81d8c30

Currently I have a test that iterates the same request 1000s of times to detect and or stimulate errors form an endpoint. The responses returned are on the order of 10s of MB each. Logging each successful attempt only seems to result in higher ram usage.

My goal would be to clear the response to null if all tests pass and if an exception occurs then log the failed response to the report.

Thanks,
Russeree

Mockups or Images of the feature

None

@russeree russeree added the enhancement New feature or request label Oct 10, 2024
@russeree russeree changed the title res.setBody() res.setBody() - Enable custom final responses. Oct 10, 2024
@helloanoop
Copy link
Contributor

Thanks for reporting this @russeree !

@lohxt1 looks like the setBody() function does not have shims in the vm. Can you look into this?

@helloanoop helloanoop added bug Something isn't working and removed enhancement New feature or request labels Oct 13, 2024
lohxt1 added a commit to lohxt1/bruno that referenced this issue Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants