Stack notification throttle checks 30 minutes but the cache entry expires after 15
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 73/100
Direzione di ricerca
Leggi src/Exceptionless.Core/Jobs/EventNotificationsJob.cs, in particolare il confronto di 30 minuti alle righe 81–83 e la scadenza della cache alle righe 150–153. Individua poi i test del job e aggiungi una copertura per il limite dell’intervallo di throttling. Il lavoro è concluso quando il confronto e la scadenza della cache usano la stessa durata, il commento corrisponde e il test conferma che le notifiche rispettano tale intervallo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
EventNotificationsJob is meant to send at most one notification per stack every 30 minutes after the first two occurrences. The throttle check compares against 30 minutes, but the cache entry recording the last send expires after 15, so the throttle lasts 15 minutes.
Where
- Comment and check use 30 minutes: EventNotificationsJob.cs#L81-L83
- The last-sent timestamp is cached for 15 minutes: EventNotificationsJob.cs#L150-L153
What happens
After a notification is sent for a stack, notify:stack-throttle:{stackId} expires 15 minutes later. The next occurrence after that reads DateTime.MinValue, skips the 30-minute comparison, and sends another notification. Repeat notifications for a busy stack therefore arrive every 15 minutes.
Expected
One window for both. The public docs currently describe 15 minutes, which is the effective behavior.
Suggested fix
Use a single TimeSpan constant for both the comparison and the cache expiration, choose 15 or 30 minutes, update the comment, and add a job test that covers the window boundary.
- Lingua principale
- C#
- Stelle
- 2.5k
- Fork
- 508
- Merge medio
- 2g 17h
- PR unite (30g)
- 40
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di exceptionless/Exceptionless
-
bug documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
exceptionless/Exceptionless#2707 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
exceptionless/Exceptionless#2706 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
exceptionless/Exceptionless#2705 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
exceptionless/Exceptionless#2704 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 1/5 1-3 ore Idoneità per principianti 88/100
exceptionless/Exceptionless#2701 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di exceptionless/Exceptionless
Issue simili
-
possible bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Include host.wslc.internal in the HTTPS development certificateForse già presa @afscrome l’ha presa oggi. Apertaarea-commandlinetools
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
dotnet/aspnetcore#70122 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
:1st_place_medal: good first issue :cowboy_hat_face: refactor :roller_coaster: enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
ApplETS/Backend-Hello#78 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 2 giorni
-
Custom component in Summary2 no longer receives raw formData (regression from app-frontend v4)Apertastatus/triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Altinn/altinn-studio#21642 ·
I maintainer di solito rispondono entro 1 giorno