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.

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

?location parameter in URLs can only point to a folder #1305

Merged
merged 5 commits into from
Oct 21, 2023
Merged

?location parameter in URLs can only point to a folder #1305

merged 5 commits into from
Oct 21, 2023

Commits on Oct 20, 2023

  1. "Hardcode" the knowledge that location= option in URL can only point … 

    …to the folder

Per @AlmightyYakob observation in
dandi/dandi-archive#1546 (comment)
    @yarikoptic
    yarikoptic committed Oct 20, 2023
    Configuration menu
    Copy the full SHA
    699a44b View commit details
    Browse the repository at this point in the history

  2. Fix

    @jwodder @yarikoptic
    jwodder authored and yarikoptic committed Oct 20, 2023
    Configuration menu
    Copy the full SHA
    de4a7f4 View commit details
    Browse the repository at this point in the history

  3. Update docs

    @jwodder @yarikoptic
    jwodder authored and yarikoptic committed Oct 20, 2023
    Configuration menu
    Copy the full SHA
    a1b23f8 View commit details
    Browse the repository at this point in the history

  4. Preserve URL parse parameters when recursing

    @jwodder @yarikoptic
    jwodder authored and yarikoptic committed Oct 20, 2023
    Configuration menu
    Copy the full SHA
    00f41ba View commit details
    Browse the repository at this point in the history

  5. RF: condense the code and thus make (internal var) location to be a n… 

    …on-Optional str
    @yarikoptic
    yarikoptic committed Oct 20, 2023
    Configuration menu
    Copy the full SHA
    24901a9 View commit details
    Browse the repository at this point in the history