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

Stop relying on girder.hub.yt for Binder #10360

Closed
pdurbin opened this issue Mar 8, 2024 · 0 comments · Fixed by #10361
Closed

Stop relying on girder.hub.yt for Binder #10360

pdurbin opened this issue Mar 8, 2024 · 0 comments · Fixed by #10361
Milestone

Comments

@pdurbin
Copy link
Member

pdurbin commented Mar 8, 2024

Currently, when you click the Binder button on a dataset with that external tool installed (e.g. my dataset on Harvard Dataverse)...

Screenshot 2024-03-08 at 11 11 45 AM

... you get an error:

{
    "message": "An unexpected error occurred on the server.",
    "type": "internal"
}

I reached out the to person who runs this service at girder.hub.yt and they said we should deploy our own equivalent solution that transforms https://girder.hub.yt/api/v1/ythub/dataverse?datasetPid=doi:10.7910/DVN/TJCLKP&siteUrl=https://dataverse.harvard.edu to https://mybinder.org/v2/dataverse/10.7910/DVN/TJCLKP/ for example. It makes sense. We never intended for it to be a long term solution.

Short term, perhaps we could create and deploy and equivalent rewrite service and adjust our guides...

Screenshot 2024-03-08 at 11 10 38 AM

... to point at docs we write for the new service. Currently, our docs point here:

Longer term (or as an alternative), we could work on this issue:

That way, we wouldn't need to rely on any rewrite service at all. Our external tool framework would be able to construct the URLs that mybinder requires with the DOI in the path instead of in a query parameter.

@pdurbin pdurbin moved this to In Progress 💻 in IQSS Dataverse Project Mar 11, 2024
@pdurbin pdurbin self-assigned this Mar 11, 2024
pdurbin added a commit that referenced this issue Mar 11, 2024
@pdurbin pdurbin added this to the 6.2 milestone Mar 11, 2024
@pdurbin pdurbin moved this from In Progress 💻 to Merged 🚀 in IQSS Dataverse Project Mar 12, 2024
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Mar 12, 2024
@pdurbin pdurbin removed their assignment May 3, 2024
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 a pull request may close this issue.

1 participant