Skip to content

lib,mod: handle map[any]any and []any from cel-go v0.27.0#122

Merged
efd6 merged 1 commit into
devfrom
string_maps
Mar 26, 2026
Merged

lib,mod: handle map[any]any and []any from cel-go v0.27.0#122
efd6 merged 1 commit into
devfrom
string_maps

Conversation

@efd6

@efd6 efd6 commented Mar 24, 2026

Copy link
Copy Markdown
Collaborator

cel-go v0.27.0 (cel-expr/cel-go#1261) changed ConvertToNative to remap interface{} targets to map[any]any for maps and []any for lists. This breaks type switches in makeMapStrings, makeStrings, and mapStrings that expected the previous map[ref.Val]ref.Val and []ref.Val types.

Add cases for the new types, round-tripping through NativeToValue and ConvertToNative with an explicit target type.

Retract v1.25.0 which shipped without this fix.

This will be tagged as v1.25.1, tested against filebeat cel input tests.

cel-go v0.27.0 (cel-expr/cel-go#1261) changed ConvertToNative to
remap interface{} targets to map[any]any for maps and []any for
lists. This breaks type switches in makeMapStrings, makeStrings,
and mapStrings that expected the previous map[ref.Val]ref.Val and
[]ref.Val types.

Add cases for the new types, round-tripping through NativeToValue
and ConvertToNative with an explicit target type.

Retract v1.25.0 which shipped without this fix.
@efd6 efd6 requested a review from andrewkroh March 24, 2026 23:42
@efd6 efd6 self-assigned this Mar 24, 2026

@ShourieG ShourieG left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@efd6 efd6 merged commit ce37f34 into dev Mar 26, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants