authzed / authzed-node

Official SpiceDB client library for NodeJS
https://authzed.com/docs/reference/api
Apache License 2.0
47 stars 18 forks source link

Error in writestream does not throw error #83

Open chuyouchia opened 6 months ago

chuyouchia commented 6 months ago

Hi there, thanks for creating the node-client! I was implementing Authzed at work and was trying to get the experimental service's bulkImportRelationships to throw errors when it fails. Specifically, I was trying to throw an error while using a Writestream created using the client library v0.13.0. image

^ In the callbacks I actually implemented callbacks in a bunch of places to try and get the error to appear.

However, it seems like these errors do not propagate (or thrown) up to the context where they have been called, even though the code execution managed to log the error. Screenshot attached below:

image

chuyouchia commented 6 months ago

I am not sure if I am using the library properly, would greatly appreciate any clarifications on this matter!