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

Cherry-pick #15557 to 7.5: Fix convert processor conversion of string with leading zeros to integer #15567

Merged
merged 1 commit into from
Jan 15, 2020

Conversation

adriansr
Copy link
Contributor

Cherry-pick of PR #15557 to 7.5 branch. Original message:

The conversion failed when for strings with leading zeros and a decimal digit 8 or 9, as the underlying runtime function would try to parse that as an octal number.

This is fixed by only allowing decimal and hex, which in turns makes the processor more aligned to its Elasticsearch counterpart.

Fixes #15513

…ger (elastic#15557)

The conversion failed when for strings with leading zeroes and a decimal
digit 8 or 9, as the underlying runtime function would try to parse that
as an octal number.

This is fixed by only allowing decimal and hex, which in turns makes the
processor more aligned to its Elasticsearch counterpart.

Fixes elastic#15513

(cherry picked from commit e0071b5)
@odacremolbap odacremolbap self-requested a review January 15, 2020 10:52
@adriansr adriansr merged commit 581e2b0 into elastic:7.5 Jan 15, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
…ger (elastic#15557) (elastic#15567)

The conversion failed when for strings with leading zeroes and a decimal
digit 8 or 9, as the underlying runtime function would try to parse that
as an octal number.

This is fixed by only allowing decimal and hex, which in turns makes the
processor more aligned to its Elasticsearch counterpart.

Fixes elastic#15513

(cherry picked from commit e26a260)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants