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

need to update NCBI URLs #18

Open

deepakunni3 opened this issue Dec 10, 2018 · 1 comment

Collaborator

deepakunni3 commented Dec 10, 2018 •

edited

Loading

All of the NCBI services have upgraded to using HTTPS but several of the URLs in the context JSON-LD are still using HTTP URL.

This results in no CURIE returned when contracting a URI using curie_util.contract_uri().

For example,

# URI with https yields no result
curie_util.contract_uri('https://www.ncbi.nlm.nih.gov/gene/9628')
[] 
# URI with http yields the correct CURIE
curie_util.contract_uri('http://www.ncbi.nlm.nih.gov/gene/9628')
['NCBIGene:9628']

But it's not clear ahead of time if all the incoming URIs are HTTP or HTTPS.

The text was updated successfully, but these errors were encountered:

deepakunni3 changed the title ~~need to update (or enumerate) NCBI URLs~~ need to update NCBI URLs

Collaborator Author

deepakunni3 commented Dec 10, 2018

This issue came up with monarch_context.jsonld.
Perhaps we have to update this JSON-LD every time Monarch makes a new release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment