Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Sep 11, 2024

Proposed commit message

The bitSize argument in strconv.ParseFloat should be either 32 or 64.
Any value other than 32 will default to 64 in the current implementation,
though this behavior is undocumented, making it undefined.

See https://cs.opensource.google/go/go/+/refs/tags/go1.23.0:src/strconv/atof.go;l=704-708.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.
    This is an automatic backport of pull request fix invalid bitSize in strconv.ParseFloat #40753 done by Mergify.

The bitSize argument in strconv.ParseFloat should be either 32 or 64.
Any value other than 32 will default to 64 in the current implementation,
though this behavior is undocumented, making it undefined.

See https://cs.opensource.google/go/go/+/refs/tags/go1.23.0:src/strconv/atof.go;l=704-708.

(cherry picked from commit 6e93c27)
@mergify mergify bot requested a review from a team as a code owner September 11, 2024 21:08
@mergify mergify bot added the backport label Sep 11, 2024
@mergify mergify bot requested review from AndersonQ and VihasMakwana and removed request for a team September 11, 2024 21:08
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 11, 2024
@mauri870 mauri870 closed this Sep 12, 2024
@mergify mergify bot deleted the mergify/bp/8.x/pr-40753 branch September 12, 2024 11:42
@pierrehilbert pierrehilbert restored the mergify/bp/8.x/pr-40753 branch September 16, 2024 15:31
@pierrehilbert pierrehilbert reopened this Sep 16, 2024
@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team label Sep 16, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Sep 16, 2024
@mauri870 mauri870 requested review from mauri870 and removed request for AndersonQ and VihasMakwana September 17, 2024 11:53
@pierrehilbert pierrehilbert merged commit ab9c232 into 8.x Sep 17, 2024
@pierrehilbert pierrehilbert deleted the mergify/bp/8.x/pr-40753 branch September 17, 2024 12:11
@khushijain21 khushijain21 mentioned this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants