Skip to content

Trim mount point before matching in encryption code - #32690

Merged
blizzz merged 2 commits into
masterfrom
bugfix/noid/fix-mountpoint-matching-encryption
Jun 2, 2022
Merged

Trim mount point before matching in encryption code#32690
blizzz merged 2 commits into
masterfrom
bugfix/noid/fix-mountpoint-matching-encryption

Conversation

@PVince81

@PVince81PVince81 commented Jun 1, 2022

Copy link
Copy Markdown
Member

Often times the mount point has a leading slash.
This fix sanitizes it to make sure matching works.

  • add unit test for this function

@PVince81

Copy link
Copy Markdown
MemberAuthor

/backport to stable24

@PVince81

Copy link
Copy Markdown
MemberAuthor

/backport to stable23

@PVince81

Copy link
Copy Markdown
MemberAuthor

/backport to stable22

@PVince81

Copy link
Copy Markdown
MemberAuthor

I've added unit tests for the touched method

Often times the mount point has a leading slash.
This fix sanitizes it to make sure matching works.
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
@PVince81
PVince81force-pushed the bugfix/noid/fix-mountpoint-matching-encryption branch from 08495ca to b212e8dCompareJune 1, 2022 16:11
@PVince81
PVince81 requested review from Pytal, artonge and blizzzJune 2, 2022 11:46
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@PVince81@blizzz@CarlSchwan