GoogleChrome / GoogleChrome/webstatus.dev

[ENHANCEMENT] Pass Context down when using Visitor for EventSummary

Open
#2,132 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Go
Stars
254
Forks
62
Avg merge
1d 10h
Merged PRs (30d)
64

Description

When parsing the EventSummary, we don't pass down the context which could be helpful when shutting down the service to cancel outstanding requests to things like the database.

https://github.com/GoogleChrome/webstatus.dev/blob/b508c57dbf5ccc276826fa623219cbb952faa796/lib/workertypes/types.go#L75-L106

We should pass the context down

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.