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

Parsing of query parameters from curl #3924

Open
1 task done
glhfuck opened this issue Feb 2, 2025 · 5 comments · May be fixed by #3931
Open
1 task done

Parsing of query parameters from curl #3924

glhfuck opened this issue Feb 2, 2025 · 5 comments · May be fixed by #3931
Assignees
Labels
enhancement New feature or request

Comments

@glhfuck
Copy link

glhfuck commented Feb 2, 2025

I have checked the following:

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

Describe the feature you want to add

Parsing of query parameters from curl.
Insomnia for example can do this.

Mockups or Images of the feature

...

@glhfuck glhfuck added the enhancement New feature or request label Feb 2, 2025
@sanish-bruno
Copy link
Collaborator

This seems to be something important, considering the ux improvement it can provide

@sanish-bruno sanish-bruno self-assigned this Feb 3, 2025
@andrei-korshikov
Copy link

Exactly the same: #3156 (comment)

@sanish-bruno Pure URL string (i.e. if I remove curl and quotes) is parsed correctly. So, it is already implemented somewhere in Bruno code:) should be easy to fix)

@anusree-bruno anusree-bruno self-assigned this Feb 4, 2025
@sanish-bruno sanish-bruno linked a pull request Feb 4, 2025 that will close this issue
5 tasks
@sanish-bruno
Copy link
Collaborator

@andrei-korshikov i just passed the raw url as it is, while creating a new http request from curl, that seems to do the job. also linked concerned pr. also checking the issue you have tagged.

@sanish-bruno
Copy link
Collaborator

@andrei-korshikov went through the other ISSUE you mentioned AND the PR associated with it, which handle encoded value, we are in discussion with it. currently with that PR, we are decoding it and showing it, upon edit, the url get broken. changes here, will need a deeper discussion, on how to handle encoded params.

@sanish-bruno
Copy link
Collaborator

have added temporary fix for #3156 for now in the pr #3931

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants