fix: Use sessionId instead of userId to prevent path traversal#501
Open
lettimepassby wants to merge 1 commit into
Conversation
Member
|
之前为了避免单个用户重复上传文件,利用当前用户ID当作共享盘。为保持历史行为兼容,可以新增一个共享策略配置。默认策略为单用户共享;可选配置为单会话共享,用于按会话隔离共享状态。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
当用户同时连接Windows机器A与Windows机器B时存在可能的权限逃逸问题,此提交将原先基于用户ID生成共享盘修改为基于会话ID
具体问题表现:
机器A仅有文件上传权限,机器B有所有权限
用户可以通过将机器A的文件放入共享盘,并在机器B中操作拖入Download文件触发文件下载即可成功在没有机器A的下载权限时发起文件下载