Skip to content

fix(rum): unmarshalling of user_action_properties#989

Merged
d0weinberger merged 1 commit intomainfrom
fix/calculated_web_metric_user_properties_unmarshalling
Feb 23, 2026
Merged

fix(rum): unmarshalling of user_action_properties#989
d0weinberger merged 1 commit intomainfrom
fix/calculated_web_metric_user_properties_unmarshalling

Conversation

@d0weinberger
Copy link
Collaborator

@d0weinberger d0weinberger commented Feb 23, 2026

Why this PR?

The user_action_properties field of dynatrace_calculated_web_metric could not be unmarshalled.
The following error log occurred:
[WARN] **web.UserActionProperties user_action_properties NOT covered

What has changed and How?

  • user_action_properties changed from TypeSet to TypeList
  • Receiver for MarshalHCL of UserActionProperties changed to pointer receiver
  • UserActionFilter now has UserActionProperties instead of *UserActionProperties

How is it tested?

Manual testing and E2E test was added.

How does it affect users?

Users can use the user_action_properties field of the dynatrace_calculated_web_metric now.

Issue: CA-18231

@d0weinberger d0weinberger marked this pull request as ready for review February 23, 2026 11:15
@d0weinberger d0weinberger force-pushed the fix/calculated_web_metric_user_properties_unmarshalling branch from 93a7dbc to e5a3e2e Compare February 23, 2026 11:16
@Kirdock
Copy link
Collaborator

Kirdock commented Feb 23, 2026

Seems like the test/example panics

Invalid address to set: []string{"user_action_filter", "0", "user_action_properties", "0", "from"}

Could you also take care of the TypeSet bug for the user_action_properties => property or follow-up?

@d0weinberger d0weinberger force-pushed the fix/calculated_web_metric_user_properties_unmarshalling branch from e5a3e2e to 2bd4f0b Compare February 23, 2026 12:33
@d0weinberger d0weinberger force-pushed the fix/calculated_web_metric_user_properties_unmarshalling branch from 2bd4f0b to ca0cf23 Compare February 23, 2026 12:49
@d0weinberger d0weinberger merged commit d74b2f5 into main Feb 23, 2026
25 of 26 checks passed
@d0weinberger d0weinberger deleted the fix/calculated_web_metric_user_properties_unmarshalling branch February 23, 2026 15:35
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