A credit can be added to a billing account that is being deleted
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 75/100
Línea de trabajo
Read CreateEntry and the billing account delete change referenced in #1999 to understand the existing transaction and locking behavior. Add the live customer row read with FOR KEY SHARE inside the transaction before the balance check and insert, returning customer.ErrNotFound for a soft deleted row. Confirm the lock coordinates with the delete's FOR UPDATE and that the account's transactions are included in its forfeit totals.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
CreateEntry checks that the billing account is live before it opens its transaction. The credit insert inside that transaction does not read the account again.
If the account delete commits between the check and the insert, a new live transaction is written for a deleted account. The delete only hides the transactions that exist when it runs, so this one is left outside the forfeit totals.
Proposed fix: inside CreateEntry's transaction, read the live customer row with FOR KEY SHARE before the balance check and the insert. It only has to conflict with the FOR UPDATE lock that the account delete takes. Return customer.ErrNotFound when the row is already soft deleted.
Context: found while reviewing the billing account delete change in #1999. It is listed there as a known gap.
- Lenguaje dominante
- Go
- Estrellas
- 344
- Forks
- 48
- Merge medio
- 2 d 10 h
- PR fusionados (30 d)
- 60
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de raystack/frontier
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 1-2 días Aptitud para principiantes 55/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 42/100
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
Los mantenedores suelen responder en 1 día
Todos los issues de raystack/frontier
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
blinklabs-io/dingo#5636 ·
Los mantenedores suelen responder en 1 día
-
automated-analysis bug cookie safe-outputs tool-improvement workflow-issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
github/gh-aw#71952 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
JetBrains/youtrack-cli#130 ·
Los mantenedores suelen responder en 1 día
-
Cloudflare Pages secret sync throws TypeError when deployment_configs[environment] is undefinedAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Infisical/infisical#8892 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
windvane requests one more page than it needsPosiblemente ocupada @TassioSales la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
projectdiscovery/subfinder#1979 ·
Los mantenedores suelen responder en 3 días