[PR #521] [MERGED] feat: higher perm auth path shadows lower one #632

Closed
opened 2026-04-08 16:53:44 +03:00 by zhus · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/sigoden/dufs/pull/521
Author: @sigoden
Created: 1/2/2025
Status: Merged
Merged: 1/2/2025
Merged by: @sigoden

Base: mainHead: feat


📝 Commits (1)

  • 8d20e08 feat: higher perm auth path shadows lower one

📊 Changes

3 files changed (+67 additions, -33 deletions)

View changed files

📝 src/auth.rs (+61 -26)
📝 src/server.rs (+2 -2)
📝 tests/auth.rs (+4 -5)

📄 Description

In /:rw;/path1:ro, the /:rw have higher perms, it shadow /path1:ro, so that /path1 is granted read and write permissions.

close #519


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/sigoden/dufs/pull/521 **Author:** [@sigoden](https://github.com/sigoden) **Created:** 1/2/2025 **Status:** ✅ Merged **Merged:** 1/2/2025 **Merged by:** [@sigoden](https://github.com/sigoden) **Base:** `main` ← **Head:** `feat` --- ### 📝 Commits (1) - [`8d20e08`](https://github.com/sigoden/dufs/commit/8d20e08796f5f604d8d2faed0305b38e7d9332e3) feat: higher perm auth path shadows lower one ### 📊 Changes **3 files changed** (+67 additions, -33 deletions) <details> <summary>View changed files</summary> 📝 `src/auth.rs` (+61 -26) 📝 `src/server.rs` (+2 -2) 📝 `tests/auth.rs` (+4 -5) </details> ### 📄 Description In `/:rw;/path1:ro`, the `/:rw` have higher perms, it shadow `/path1:ro`, so that `/path1` is granted read and write permissions. close #519 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
zhus added the pull-request label 2026-04-08 16:53:44 +03:00
zhus closed this issue 2026-04-08 16:53:44 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sigoden/dufs#632