honeycombio / libhoney-go

Go library for sending data to Honeycomb
Apache License 2.0
58 stars 40 forks source link

feat: support classic-flavored ingest keys #237

Closed jharley closed 7 months ago

jharley commented 7 months ago

Which problem is this PR solving?

We've now released Ingest Keys, but in order for them to work with Classic environments properly we need to update the key detection logic.

Short description of the changes

jharley commented 7 months ago

Second, if you check length manually and then remove the counts from the regexp, it runs literally 1000x faster (7 nS instead of 7 uS!)

Whoa! Thank you for this: that's huge. Will refactor the shape of the function