TS SDK: failed procedure calls reject with a bare string instead of an Error
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
- 82/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- typescript
- Ambito
- devtools
Direzione di ricerca
Inizia da crates/bindings-typescript/src/sdk/db_connection_impl.ts, nei callback di procedura di #callProcedureWithEncodedName e #callProcedureGeneric, dove il ramo di errore rifiuta con la stringa grezza result.value. Confronta con il percorso dei reducer nello stesso file, che racchiude il valore in una sottoclasse di Error. È completato quando entrambi i percorsi di procedura rifiutano con un Error che porta lo stesso messaggio e un controllo Jest rejects.toThrow su una chiamata di procedura fallita supera.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Version: spacetimedb npm 2.11.0 and 2.12.0; still present on master (1b67ca3).
Where: crates/bindings-typescript/src/sdk/db_connection_impl.ts, the procedure callbacks in #callProcedureWithEncodedName and #callProcedureGeneric:
if (result.tag === 'Ok') {
resolve(result.value);
} else {
reject(result.value); // a plain string
}
Reducer calls in the same file reject with new InternalError(result.value) (or SenderError), so await conn.reducers.x(...) always rejects with an Error, but await conn.procedures.x(...) rejects with a bare string when the host fails the call (for example invalid arguments for procedure ...: data too short ... after a module republish changes a parameter type).
Effect:
- No stack trace, and
error instanceof Errorchecks treat it as unknown. - Jest's
await expect(p).rejects.toThrow(...)reports "Received function did not throw" for a rejected procedure call.
Suggested fix: reject with new InternalError(result.value) in both procedure callbacks, matching the reducer path.
- Lingua principale
- Rust
- Stelle
- 25.3k
- Fork
- 1.1k
- Merge medio
- 3g 31m
- PR unite (30g)
- 66
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Nessuna 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 clockworklabs/SpacetimeDB
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
clockworklabs/SpacetimeDB#6314 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
TypeScript server `ctx.random()`: first draw of every transaction is heavily biased (signed shift in `pcg32` seeding)Forse già presa @Tyagiquamar l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
clockworklabs/SpacetimeDB#6247 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
clockworklabs/SpacetimeDB#6146 ·
I maintainer di solito rispondono entro 1 giorno
-
CI flake: LintsAperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
clockworklabs/SpacetimeDB#5985 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
clockworklabs/SpacetimeDB#5943 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di clockworklabs/SpacetimeDB
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
EmbarkStudios/cargo-deny#910 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
tursodatabase/turso#10054 ·
I maintainer di solito rispondono entro 1 giorno
-
build: Update rust versionForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertadependencies
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
ibm-granite-community/granite-cli#229 ·
I maintainer di solito rispondono entro 1 giorno
-
setTargetAtTime booked ahead of time starts its curve early (values blow up before start_time)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
orottier/web-audio-api-rs#658 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
aosama/astronomical#1318 ·
I maintainer di solito rispondono entro 1 giorno