Emily S

Results 72 comments of Emily S

Hi @janko thanks for this contribution! As we would continue to support older versions of `graphql`, do these changes support `.trace_with` in addition to `.trace` or do the changes replace...

Hey @janko I think some of the GraphQL tests are failing, would you mind taking a look? For example: https://github.com/elastic/apm-agent-ruby/actions/runs/8418659477/job/23206725271?pr=1446#step:4:692

Hi @kurigashvili-godaddy thanks for opening this issue. It seems reasonable to support passing the trace context to the sidekiq job. Since you've already identified all the relevant parts of the...

Hi @jclusso I made these changes hoping they would fix the issues you were seeing but when we tested, they didn't. So I am hesitant to merge these changes without...

I'm going to hold off adding a fix for this in the next gem version, as @mikker and I didn't come to a decision. When he returns, we can decide...

Hi @bscharm! Thank you for bringing up this shortcoming of reporting errors with async spans and for the detailed description. From a high level, the solution you proposed sounds reasonable....

Hi again @bscharm I opened a PR (#975) to fix this. I have one outstanding question, which you can see in the PR description. Perhaps you have an opinion on...

Hi @bscharm thanks for this update and trying out the PR. I'll do some more testing and have more information next week.

@dgoradia Thank you for reporting this. We'll have an update asap. In the meantime, can you tell me which field of the transaction metadata contained the `RAW_POST_DATA`? And if I'm...