Skip to content

ref(grouping): Correctly parameterize dotted ints#114776

Merged
lobsterkatie merged 2 commits intomasterfrom
kmclb-fix-float-and-ip-parameterization-regexes-to-only-have-two-or-four-parts
May 4, 2026
Merged

ref(grouping): Correctly parameterize dotted ints#114776
lobsterkatie merged 2 commits intomasterfrom
kmclb-fix-float-and-ip-parameterization-regexes-to-only-have-two-or-four-parts

Conversation

@lobsterkatie
Copy link
Copy Markdown
Member

This fixes our IPv4 and float parameterization regexes so that IPv4 values must have exactly 4 dotted segments, and floats must have exactly 2. (Currently, 1.2.3.4.5 matches as <ip>.<int> and 1.2.3 matches as <float>.<int>.)

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 4, 2026
@lobsterkatie lobsterkatie marked this pull request as ready for review May 4, 2026 21:12
@lobsterkatie lobsterkatie requested a review from a team as a code owner May 4, 2026 21:12
@lobsterkatie lobsterkatie changed the title ref(grouping): Correctly parameterize dotted ints and ints ref(grouping): Correctly parameterize dotted ints May 4, 2026
@lobsterkatie lobsterkatie merged commit 636652c into master May 4, 2026
80 checks passed
@lobsterkatie lobsterkatie deleted the kmclb-fix-float-and-ip-parameterization-regexes-to-only-have-two-or-four-parts branch May 4, 2026 21:24
cleptric pushed a commit that referenced this pull request May 5, 2026
This fixes our IPv4 and float parameterization regexes so that IPv4 values must have exactly 4 dotted segments, and floats must have exactly 2. (Currently, `1.2.3.4.5` matches as `<ip>.<int>` and `1.2.3` matches as `<float>.<int>`.)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants