google / exposure-notifications-server

Exposure Notification Reference Server | Covid-19 Exposure Notifications
https://www.google.com/covid19/exposurenotifications/
Apache License 2.0
2.43k stars 311 forks source link

Use timeout instead of passing context in o11y exporter #1544

Closed sethvargo closed 3 years ago

sethvargo commented 3 years ago

Per the documentation, the provided context must not have a timeout, but we're consistently seeing context deadline exceeded errors. I think this is because we're passing in a context that has a timeout. I haven't been able to find a place where we actually call it with a timeout context, but either way, we shouldn't use the provided context on all requests since it comes from the outside. There's also already a timeout set on the export requests through the Timeout options property.

Release Note

NONE