Skip to content
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

Remove webhooks for anything but alerts #1639

Open
ChrisSchinnerl opened this issue Oct 30, 2024 · 0 comments
Open

Remove webhooks for anything but alerts #1639

ChrisSchinnerl opened this issue Oct 30, 2024 · 0 comments
Milestone

Comments

@ChrisSchinnerl
Copy link
Member

ChrisSchinnerl commented Oct 30, 2024

With v2 we can get rid of webhooks for cache invalidation. I had a discussion with @n8maninger and we came to the conclusion that it's fine to only rely on short (e.g. 5 minute) caching for contracts/hosts.

Gouging settings should be polled every time they are needed. However, we only need them on upload within the worker. For downloads we can consider them good enough at the time we fund the account, which we do in the bus now.

@ChrisSchinnerl ChrisSchinnerl converted this from a draft issue Oct 30, 2024
@ChrisSchinnerl ChrisSchinnerl changed the title Remove webhooks Remove webhooks for anything but alerts Oct 30, 2024
@ChrisSchinnerl ChrisSchinnerl moved this from Triage to Todo in Sia Oct 30, 2024
@ChrisSchinnerl ChrisSchinnerl added this to the v2.0.0 milestone Oct 30, 2024
@n8maninger n8maninger removed their assignment Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

2 participants