feat: support for sendRequest functionality #3845
Conversation
| @@ -153,6 +155,63 @@ class Bru { | |||
| sleep(ms) { | |||
| return new Promise((resolve) => setTimeout(resolve, ms)); | |||
| } | |||
|
|
|||
| sendRequest(requestConfig, callback) { | |||
There was a problem hiding this comment.
Would it be possible to send the request and pass a specific variable to be used in either the pre or post scripts?
For instance, I'd like to call bru.sendRequest('./items/delete-item', { itemId: myCustomItemId }). This would allow developers to customize further their scripts by replacing "request-scoped" variables
There was a problem hiding this comment.
@thiagomini Bruno provides an API, bruno.runRequest('items/delete-item'), to trigger another request already defined in Bruno.
The sendRequest() API, on the other hand, is used for making direct HTTP requests.
It seems logical to support passing variables when using sendRequest(). Could you please create a new GitHub issue outlining this requirement?
There was a problem hiding this comment.
Oh, I see @helloanoop , thanks for the info! So, my idea is actually related to passing variables using the bruno.runRequest, do we have that? If not, I'll create an issue asking for that feature
|
Looks great! Just curious, shouldn't the proxy be applied to the sendRequest as well? |
Description
This PR introduces support for the
sendRequestfunction, enhancing the framework's ability to handle HTTP requests dynamically. The implementation supports multiple HTTP methods (GET,POST,PUT,PATCH,DELETE) and allows Callback-based execution.Key Changes:
sendRequestfunction with support for:Contribution Checklist: