Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

[Audit] Fee-collection events are the only handler not wrapped in a transaction, and worker inconsistency compounds UX-visible drift

オープン
#1,239 コメント 6 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 7 日以内に返信

@d1a-m3n がすでに取り組んでいます。

2026年8月29日 から。

評価

この issue はまだ評価されていません。

説明

backend enhancement second-wave-audit Stellar Wave ux

Filed from the second-wave repository audit (issue 37/100). See the audit summary for full category context.

  • Location: backend/src/workers/soroban-event-worker.ts:1000-1060 (handleFeeCollected)
  • Problem: Unlike every other event handler in the file, this one performs its dedup check and its upsert as two separate top-level prisma calls instead of one prisma.$transaction, reopening a TOCTOU window the rest of the file was clearly designed to avoid.
  • Evidence: No $transaction wrapper, in contrast to sibling handlers in the same file.
  • Suggested implementation: Wrap the dedup-check + upsert in prisma.$transaction, matching the other handlers.
  • Acceptance criteria: A test simulating two near-simultaneous fee-collected events for the same tx confirms exactly one StreamEvent row is created.
  • Difficulty: S
  • Expected impact: Closes a duplicate-fee-event race visible to anyone reconciling fee revenue from the API.
主要言語
TypeScript
スター
18
フォーク
250
平均マージ
5日 6時間
マージ済み PR(30日)
83

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

LabsCrypt/flowfi のほかの issue

LabsCrypt/flowfi の issue をすべて見る

似ている issue

TypeScript の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。