Ensure unicode characters are interpreted as desired

snowplow / snowplow-golang-analytics-sdk

Golang Analytics SDK for working with Snowplow enriched events in cloud functions and other Go applications.

https://snowplowanalytics.com/

Apache License 2.0

4 stars 1 forks source link

Closed colmsnowplow closed 2 years ago

colmsnowplow commented 2 years ago

In https://github.com/snowplow/snowplow-golang-analytics-sdk/issues/4 we made a change which we believed would result in unicode characters like < being escaped. It turns out this was incorrect, because of a testing error.

We should return to this issue, but I now have new questions about it:

Should we keep escaping html characters, or revert that config?
Should we care that unicode characters are represented as unicode, if that's how go natively treats them?