New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add synthetics info header #2203
Conversation
def snakeize(string) | ||
string.gsub(/(.)([A-Z])/, '\1_\2').downcase | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! We have a language support test file. Could you add tests for this method to it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added a test for this here 436f5d2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the name of this method 🐍
Co-authored-by: Kayla Reopelle (she/her) <87386821+kaylareopelle@users.noreply.github.com>
SimpleCov Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
This adds the logic for handling the new synthetics header,
X-NewRelic-Synthetics-Info
, adding the attributes to the transaction as intrinsic attributes, and adds the header to outgoing external request headers. These attributes are included in transaction events and error events. They will also automatically be included in transaction/error traces, as those include all intrinsic attributes already.spec
closes #2055