Skip to content

[stable27] add some support for rename on case insensitive local filesystems - #41502

Merged
Altahrim merged 1 commit into
stable27from
backport/40935/stable27
Nov 16, 2023
Merged

[stable27] add some support for rename on case insensitive local filesystems#41502
Altahrim merged 1 commit into
stable27from
backport/40935/stable27

Conversation

@Altahrim

Copy link
Copy Markdown
Collaborator

Manual backport of #40935

Signed-off-by: Robin Appelman <robin@icewind.nl>
@AltahrimAltahrim added the 3. to review Waiting for reviews label Nov 15, 2023
@AltahrimAltahrim added this to the Nextcloud 27.1.4 milestone Nov 15, 2023
@AltahrimAltahrim self-assigned this Nov 15, 2023
@Altahrim
Altahrim requested review from ArtificialOwl and come-nc and removed request for a teamNovember 15, 2023 14:23
}

return rename($this->getSourcePath($source), $this->getSourcePath($target));
if (rename($this->getSourcePath($source), $this->getSourcePath($target))) {

Check failure

Code scanning / Psalm

TaintedFile

Detected tainted file handling
}

return rename($this->getSourcePath($source), $this->getSourcePath($target));
if (rename($this->getSourcePath($source), $this->getSourcePath($target))) {

Check failure

Code scanning / Psalm

TaintedFile

Detected tainted file handling
@blizzzblizzz mentioned this pull request Nov 15, 2023
@Altahrim
Altahrim merged commit 902ba76 into stable27Nov 16, 2023
@Altahrim
Altahrim deleted the backport/40935/stable27 branch November 16, 2023 15:34
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Altahrim@AndyScherzinger@github-advanced-security@come-nc@icewind1991