Curl import changes URL

[allanf181]

I have checked the following:

• I use the newest version of bruno.
• I've searched existing issues and found nothing related to my issue.

Describe the bug

When a curl command is imported, the url is changed by removing the last /
curl 'http://example.com/test/' turns into http://example.com/test

This affects endpoints that end with /

.bru file to reproduce the bug

No response

Screenshots/Live demo link

Bruno_bLISVj39uA.mp4

[satokenta940]

When you import cURL from Apidog, you won't encounter this issue. Give it a try; I think it's quite effective.

[jokj624]

bruno/packages/bruno-app/src/utils/curl/curl-to-json.js

Line 126 in b01f2e0

requestJson.url = request.urlWithoutQuery.replace(/\/$/, '');

Maybe it's removed by this line.
However, is there a case where it only works when there is a '/' at the end of the url?

[jwetzell]

@jokj624 that line is actually specifically written to remove the / from the end of a url. So seems like they are intentionally stripping of that last backslash for some reason. Not sure why the raw_url field also gets the processed value. That does not seem "raw" to me.

[jkjustjoshing]

I can't imagine a situation in which I want the request that Bruno sends out to be different from the request cURL sends out. If the user doesn't want a trailing slash they would just delete it from the URL after import (or in the cURL command).

This caused me a huge headache yesterday. I was trying to hit this Python FastAPI endpoint, and I was getting a correct response using cURL on the command line, but importing into Bruno kept giving me a "405 method not allowed" error.

I ended up spinning up a webserver that echoed back the headers, thinking maybe Bruno was attaching a hidden header somewhere? I thought it would be great if Bruno had an "export to cURL" feature, so I could have done a string comparison with the cURL syntax, but I don't believe that exists in Bruno.

15 minutes later, I realized that the trailing / was being stripped off.

[jkjustjoshing]

@helloanoop do you remember why you chose to strip the trailing slash?

[helloanoop]

No, I don't remember. Agree that we should not strip the slash if curl command has it.
@sanjai0py Can you raise a PR for this ?