Passing null to addAttribute is causing a telemetry error complaining about a contract violation on the native side. The result is that the attribute wouldn't be changed. This changes it so that if null is passed to addAttribute, it calls removeAttribute instead, which will prevent the contract violation and is likely more consistent with expectations.
How?
If needed, a description of how this PR accomplishes what it does.
Review checklist
[ ] This pull request has appropriate unit and / or integration tests
[ ] This pull request references a Github or JIRA issue
What and why?
Passing
nulltoaddAttributeis causing a telemetry error complaining about a contract violation on the native side. The result is that the attribute wouldn't be changed. This changes it so that ifnullis passed toaddAttribute, it callsremoveAttributeinstead, which will prevent the contract violation and is likely more consistent with expectations.How?
If needed, a description of how this PR accomplishes what it does.
Review checklist