Skip to content

Commit eb48984

Browse files
authored
Fix issue with handling big numbers (#1599)
Most of the fix is in Roast, just verified here. Fixes #1592 Signed-off-by: Anders Eknert <anders@styra.com>
1 parent b8f3e8f commit eb48984

5 files changed

Lines changed: 17 additions & 8 deletions

File tree

bundle/regal/main/main_test.rego

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -434,3 +434,14 @@ test_aggregate_report_custom_rule if {
434434

435435
violations == report
436436
}
437+
438+
# verify fix for https://github.com/styrainc/regal/issues/1592
439+
test_big_number_causes_no_parser_error if {
440+
policy := `package p
441+
442+
big := 1e1000
443+
`
444+
module = regal.parse_module("p.rego", policy)
445+
446+
module.rules[0].head.value.value == 1e1000
447+
}

e2e/testbuild/go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ require (
9191
github.com/spf13/cobra v1.9.1 // indirect
9292
github.com/spf13/pflag v1.0.6 // indirect
9393
github.com/spf13/viper v1.20.1 // indirect
94-
github.com/styrainc/roast v0.10.0 // indirect
94+
github.com/styrainc/roast v0.11.2 // indirect
9595
github.com/subosito/gotenv v1.6.0 // indirect
9696
github.com/tchap/go-patricia/v2 v2.3.2 // indirect
9797
github.com/vektah/gqlparser/v2 v2.5.26 // indirect

e2e/testbuild/go.sum

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -275,10 +275,8 @@ github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/
275275
github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
276276
github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
277277
github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
278-
github.com/styrainc/regal v0.34.1 h1:raOR30iTI38c4MFxW0LpG0p0jTtQcIp/OtFjUn+Adic=
279-
github.com/styrainc/regal v0.34.1/go.mod h1:85wnQGlKcZmVyCqI1feZMLIT0Zds8feN3g1Pvf0Dxsc=
280-
github.com/styrainc/roast v0.10.0 h1:mD0HLezVC35qnZLa8tQYq3EFkcGxAJSddP+dBPU+7o4=
281-
github.com/styrainc/roast v0.10.0/go.mod h1:75o0XHVEbDZp456w5EzDCf3ve5tm1YrJu1SHW+NRnLg=
278+
github.com/styrainc/roast v0.11.2 h1:6sE6bme2uRC/tuC2+n6iHVDg1dLw7XCI03GGi8Tadf4=
279+
github.com/styrainc/roast v0.11.2/go.mod h1:cKELz96vKP1jeA/0HeyMW6gOqvDI6eo3wmq5/a43TYA=
282280
github.com/subosito/gotenv v1.6.0 h1:9NlTDc1FTs4qu0DDq7AEtTPNw6SVm7uBMsUCUjABIf8=
283281
github.com/subosito/gotenv v1.6.0/go.mod h1:Dk4QP5c2W3ibzajGcXpNraDfq2IrhjMIvMSWPKKo0FU=
284282
github.com/tchap/go-patricia/v2 v2.3.2 h1:xTHFutuitO2zqKAQ5rCROYgUb7Or/+IC3fts9/Yc7nM=

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ require (
2222
github.com/sourcegraph/jsonrpc2 v0.2.1
2323
github.com/spf13/cobra v1.9.1
2424
github.com/spf13/pflag v1.0.6
25-
github.com/styrainc/roast v0.10.0
25+
github.com/styrainc/roast v0.11.2
2626
gopkg.in/yaml.v3 v3.0.1
2727
)
2828

go.sum

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -305,8 +305,8 @@ github.com/stretchr/testify v1.7.1/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/
305305
github.com/stretchr/testify v1.8.0/go.mod h1:yNjHg4UonilssWZ8iaSj1OCr/vHnekPRkoO+kdMU+MU=
306306
github.com/stretchr/testify v1.10.0 h1:Xv5erBjTwe/5IxqUQTdXv5kgmIvbHo3QQyRwhJsOfJA=
307307
github.com/stretchr/testify v1.10.0/go.mod h1:r2ic/lqez/lEtzL7wO/rwa5dbSLXVDPFyf8C91i36aY=
308-
github.com/styrainc/roast v0.10.0 h1:mD0HLezVC35qnZLa8tQYq3EFkcGxAJSddP+dBPU+7o4=
309-
github.com/styrainc/roast v0.10.0/go.mod h1:75o0XHVEbDZp456w5EzDCf3ve5tm1YrJu1SHW+NRnLg=
308+
github.com/styrainc/roast v0.11.2 h1:6sE6bme2uRC/tuC2+n6iHVDg1dLw7XCI03GGi8Tadf4=
309+
github.com/styrainc/roast v0.11.2/go.mod h1:cKELz96vKP1jeA/0HeyMW6gOqvDI6eo3wmq5/a43TYA=
310310
github.com/subosito/gotenv v1.6.0 h1:9NlTDc1FTs4qu0DDq7AEtTPNw6SVm7uBMsUCUjABIf8=
311311
github.com/subosito/gotenv v1.6.0/go.mod h1:Dk4QP5c2W3ibzajGcXpNraDfq2IrhjMIvMSWPKKo0FU=
312312
github.com/tchap/go-patricia/v2 v2.3.2 h1:xTHFutuitO2zqKAQ5rCROYgUb7Or/+IC3fts9/Yc7nM=

0 commit comments

Comments
 (0)