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

Revert "Use URLSearchParams() to construct query strings" #1755

Merged
merged 1 commit into from
Oct 2, 2023

Conversation

manics
Copy link
Member

@manics manics commented Oct 2, 2023

Reverts #1741

This code isn't tested by our CI, and resulted in a failure on mybinder.org: #1747 which now blocks further BinderHub updates.

This syntax can be seen to be invalid in a browser javascript console:

>> new URL("/" + "build/")
Uncaught TypeError: URL constructor: /build/ is not a valid URL.

@manics manics added the bug label Oct 2, 2023
@manics manics requested a review from yuvipanda October 2, 2023 22:57
@yuvipanda yuvipanda merged commit 9e3ebef into main Oct 2, 2023
22 of 26 checks passed
@yuvipanda yuvipanda deleted the revert-1741-binderhub-client-urlparams branch October 2, 2023 23:31
@yuvipanda
Copy link
Collaborator

Ugh, I can see how this worked when I tested this locally but failed on mybinder.org :( I'll put up another change.

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

Successfully merging this pull request may close these issues.

2 participants