When LWP integration sends an error it puts an integer in the status description that is required to be a string.
Closed this issue · 5 comments
Fixed in #14.
When sending data through otel-collector it generates a 400 error when trying to parse the status description.
Thanks for the report!
I fee like this is an issue at the exporter level: the exporter should know to send the data in the correct format, regardless of what the integrations put in.
Are you using the JSON exporter? I think the protobuf exporter should be converting this to a string anyway, but the JSON one I don't think is giving doing anything about types (so there may be other fields we want to coerce).
Yes this is the JSON exporter, this 'fix' forces the issue but your probably right the real bug is in the json exporter.
If that's the case, I think this might be what is needed: jjatria/perl-opentelemetry-exporter-otlp@6ffd710
If you are able to check this works on your end, then I can mint a release and push to CPAN. If not... well, I guess I can still push to CPAN. 😅
I'll release before tomorrow morning.
I just pushed version 0.016 of OpenTelemetry::Exporter::OTLP, which contains this change. It should be indexed to CPAN shortly.
Please let me know if it solves your issue, so we can close this. And thanks again for raising it!
Closing this since this issue seems to have been resolved.