Skip to content

Conversation

@provokateurin
Copy link
Member

Summary

The pattern wasn't allowing - because it didn't match

if (!preg_match('/^[a-z0-9-]+$/i', $token)) {
.

Checklist

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin added bug 3. to review Waiting for reviews labels Mar 10, 2025
@provokateurin provokateurin added this to the Nextcloud 32 milestone Mar 10, 2025
@provokateurin provokateurin requested a review from Pytal March 10, 2025 09:11
@provokateurin provokateurin requested a review from a team as a code owner March 10, 2025 09:11
@provokateurin provokateurin requested review from Altahrim, icewind1991 and skjnldsv and removed request for a team March 10, 2025 09:11
@provokateurin
Copy link
Member Author

/backport to stable31

Copy link
Member

@Pytal Pytal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@provokateurin provokateurin enabled auto-merge March 11, 2025 05:09
@provokateurin provokateurin merged commit 18d5ff6 into master Mar 11, 2025
190 checks passed
@provokateurin provokateurin deleted the fix/dav/publicremote-share-token-pattern branch March 11, 2025 13:24
@skjnldsv
Copy link
Member

/backport to stable31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Using hyphens in customized share link results in error

4 participants