Skip to content

fix(paths): preserve full virtual paths for scoped users - #311

Open
okatu-loli wants to merge 1 commit into
mainfrom
codex/fix-user-base-path
Open

fix(paths): preserve full virtual paths for scoped users#311
okatu-loli wants to merge 1 commit into
mainfrom
codex/fix-user-base-path

Conversation

@okatu-loli

@okatu-loli okatu-loli commented Sep 6, 2026

Copy link
Copy Markdown
Collaborator

Scoped users can send a directory prefix twice, lose ancestor segments when following a role directory, or stay on a loading screen after a failed request. Use full virtual paths consistently and leave the user base path at /.

Changes

  • Remove role-prefix guessing from list requests and relative navigation.
  • Carry full virtual paths on synthetic role folders, including same-named folders under different parents, for list and grid navigation.
  • Finish failed requests with a visible error instead of redirecting to the same path.
  • Make the user base path read-only at / and stop deriving it from selected roles.
  • Keep login redirect URLs encoded until navigation, so spaces, # and % survive login and preview.

Validation

  • Production Vite build and formatting checks passed.
  • Browser checks passed for multilevel/same-name list and grid navigation, failed-folder error state, special-character login redirect and text preview, and the migrated role/base-path management form.
  • Installed TypeScript 4.7 stops on the pre-existing satisfies expression in src/store/obj.ts:300; that source is unchanged by this PR.

Coordinate release with AlistGo/alist#9641, which transfers non-root base restrictions to roles and keeps base paths at /.

@netlify

netlify Bot commented Sep 6, 2026

Copy link
Copy Markdown

βœ… Deploy Preview for alist-web ready!

Name Link
πŸ”¨ Latest commit 81f4f94
πŸ” Latest deploy log https://app.netlify.com/projects/alist-web/deploys/6a9d0a3199e3cd0008ad565f
😎 Deploy Preview https://deploy-preview-311--alist-web.netlify.app
πŸ“± Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
πŸ€– Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

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 this pull request may close these issues.

1 participant