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

Long numbers (integers and floats) converted to strings #2202

Open
2 tasks done
rseeley opened this issue Apr 29, 2024 · 1 comment · May be fixed by #2236
Open
2 tasks done

Long numbers (integers and floats) converted to strings #2202

rseeley opened this issue Apr 29, 2024 · 1 comment · May be fixed by #2236
Labels
bug Something isn't working

Comments

@rseeley
Copy link

rseeley commented Apr 29, 2024

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

Using the latest version of Bruno available on flatpak and brew, 1.14.0, using large/long numbers (integers and float) are converted to strings.

.bru file to reproduce the bug

meta {
  name: test long numbers
  type: http
  seq: 4
}

post {
  url: http://localhost:8000/utils/test-long-numbers
  body: json
  auth: none
}

body:json {
  {"someValue": 12345678901234.1}
}

Screenshots/Live demo link

Screenshot 2024-04-29 at 3 56 28 PM Screenshot 2024-04-29 at 1 50 15 PM Screenshot 2024-04-29 at 1 50 34 PM Screenshot 2024-04-29 at 1 50 41 PM Screenshot 2024-04-29 at 1 50 48 PM Screenshot 2024-04-29 at 1 54 19 PM Screenshot 2024-04-29 at 1 54 26 PM
@rseeley rseeley added the bug Something isn't working label Apr 29, 2024
@austenadler austenadler linked a pull request May 4, 2024 that will close this issue
5 tasks
@axelson
Copy link

axelson commented May 7, 2024

A temporary workaround for this is to set the body type to TEXT and add a Content-Type: application/json header.

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