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

Pre request variables ignored within post url #2664

Closed
2 tasks done
lrotermund opened this issue Jul 18, 2024 · 2 comments
Closed
2 tasks done

Pre request variables ignored within post url #2664

lrotermund opened this issue Jul 18, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@lrotermund
Copy link

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

Context: For good integration of Bruno in our large team, we maintain URL variables like /foo/{{foo_id}}/bar as 'Pre Request' variables, which we commit empty and do not overwrite.

Since some update that I can no longer specify, this functionality is broken and variables stored in the pre-request are no longer set and are simply submitted empty instead. Staying with the previous example

  • Post url: /foo/{{foo_id}}/bar
  • Pre-request foo_id: 123
  • Actual requested URL: /foo//bar

Was this behaviour intentionally patched?

.bru file to reproduce the bug

meta {
  name: Buggy pre variables test
  type: http
  seq: 1
}

post {
  url: https://google.com/{{api}}
  body: json
}

headers {
  Content-Type: application/json
}

body:json {
  {
    "q": "{{query}}"
  }
}

vars:pre-request {
  api:
  query:
}

Screenshots/Live demo link

  1. shows the use of the defined 'Pre Request' variable 'api'.
  2. shows that the pre-request variable 'api' has been set.
  3. shows the timeline of the request and the empty replaced variables.
  4. same as 3.
  5. shows the current bruno version v1.21.0

image

@lrotermund lrotermund added the bug Something isn't working label Jul 18, 2024
@Its-treason
Copy link
Member

Its-treason commented Jul 18, 2024

Duplicate of #2627 it should be fixed in the next version when #2651 is merged

@lrotermund
Copy link
Author

@Its-treason Thanks for the feedback, sorry I missed it when searching the open issues. I'm looking forward to the fix, as bruno is barely usable for us at the moment.

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

No branches or pull requests

3 participants