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.

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

passing meaningful information into an exception when deserialization… #141

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zdary
Copy link

@zdary zdary commented Jan 28, 2025

… goes wrong.

The current message lookend like this:
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected STRING but was BEGIN_OBJECT at path $.records[0]

We need to get more info to figure out what's the actual issue.

… goes wrong.

The current message lookend like this:
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected STRING but was BEGIN_OBJECT at path $.records[0]

We need to get more info to figure out what's the actual issue.
Copy link
Contributor

@dvacca-onfido dvacca-onfido left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @zdary , thanks for your contribution but unfortunately it can't be integrated as it's. As in the README file, this kind of change needs to be performed in the template instead since the library is auto-generated.

@zdary
Copy link
Author

zdary commented Jan 28, 2025

Thank you, I've moved this to onfido/onfido-openapi-spec#152

@zdary
Copy link
Author

zdary commented Feb 5, 2025

@dvacca-onfido Can anyone look at my PR at onfido/onfido-openapi-spec#152 ? Seem like no activity on that repo :(

@dvacca-onfido
Copy link
Contributor

@dvacca-onfido Can anyone look at my PR at onfido/onfido-openapi-spec#152 ? Seem like no activity on that repo :(

@zdary, I apologise for the delay; please bear with us, we'll be back on that very soon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants