apache / apache/pulsar-client-node

No Error handling for Deserialize method which ends up crashing Node JS

Ouverte
#422 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
C++
Étoiles
164
Forks
98
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Hello Team

Here I am trying to serialize and deserialize messageId for my business use case

but issue is for proper error handling if in case we pass invalid buffer data to MessageId.deserialize method then it throws C++ exception
`libc++abi: terminating due to uncaught exception of type std::invalid_argument: Failed to parse serialized message id`
which is not handled by pulsar-client and node JS crashes

I see similar issue was there while creating a client too but that got fixed with this issue and PR https://github.com/apache/pulsar-client-node/pull/307/files
where it was wrapped with try catch block
but deserialize method is still not fixed

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Piste de recherche

Commencez par MessageId.deserialize et comparez sa gestion des erreurs avec la gestion côté client introduite dans PR #307. Reproduisez le cas d’un buffer invalide et suivez le binding de deserialize ; le travail est terminé lorsque des entrées invalides ne terminent plus Node.js avec une exception C++ non interceptée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp, javascript, node.js
Domaine
api
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.