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

Closed
2 tasks done
rseeley opened this issue Apr 29, 2024 · 2 comments
Closed
2 tasks done

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

rseeley opened this issue Apr 29, 2024 · 2 comments
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
@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.

@helloanoop
Copy link
Contributor

This has been fixed as a part of PR #2773, and will go out in the v1.27.0 release.

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.

3 participants