Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ReponseStatus Deserialization Panics If No Message Provided #206

Closed
criminosis opened this issue Apr 18, 2024 · 1 comment · Fixed by #207
Closed

ReponseStatus Deserialization Panics If No Message Provided #206

criminosis opened this issue Apr 18, 2024 · 1 comment · Fixed by #207

Comments

@criminosis
Copy link
Contributor

This may be uniquely JanusGraph, but I've had panics that occur because the message field of ResponseStatus wasn't being returned or was null. Setting a default value of empty string can avoid the panic.
@criminosis criminosis mentioned this issue Apr 18, 2024
Merged
@criminosis
Copy link
Contributor Author

Getting ahead of the curve for this one @wolf4ood and went ahead and put up a PR 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Default message to empty string for ResponseStatus criminosis/gremlin-rs
1 participant
@criminosis