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

500 error for partial reference #31

Open
aaccomazzi opened this issue Feb 16, 2021 · 2 comments
Open

500 error for partial reference #31

aaccomazzi opened this issue Feb 16, 2021 · 2 comments
Assignees

Comments

@aaccomazzi
Copy link
Member

When using the following partial reference, the service returns a 500 error:

Accomazzi, A., Eichhorn, G., Kurtz, M.J., et al. 2000, A&AS

The corresponding API URL is https://ui.adsabs.harvard.edu/v1/reference/text/Accomazzi%2C%20A.%2C%20Eichhorn%2C%20G.%2C%20Kurtz%2C%20M.J.%2C%20et%20al.%202000%2C%20A%26AS

@golnazads
Copy link
Collaborator

sorry, there is something going on with caching that I am going to track down to see what is happening. But if I disable the caching this is what it come back as
2021-02-21T18:21:22.934Z 934 ERROR [99721:Thread-1:views.py:151] Exception: ('Not enough information to resolve the record.', <referencesrv.resolver.hypotheses.Hypotheses object at 0x151ee7af0>)
it would not even attempt it when it knows it is journal that has to have more information. that is the right thing to do, right?

@aaccomazzi
Copy link
Member Author

Correct. We should catch the exception and return the error message to the user in a properly encoded json message.

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

No branches or pull requests

2 participants