Skip to content

fix(notifications): Require absolute links for support of desktop and mobile clients - #2514

Merged
miaulalala merged 1 commit into
masterfrom
fix/sanitize-notification-urls
Jul 27, 2026
Merged

fix(notifications): Require absolute links for support of desktop and mobile clients#2514
miaulalala merged 1 commit into
masterfrom
fix/sanitize-notification-urls

Conversation

@miaulalala

Copy link
Copy Markdown
Collaborator

To align with the stricter validation introduced in nextcloud/server#59606.

AI-Assisted-By: Claude Opus 4.6 noreply@anthropic.com

@miaulalala

Copy link
Copy Markdown
Collaborator Author

/backport to stable33

@miaulalala

Copy link
Copy Markdown
Collaborator Author

/backport to stable32

@cypress

cypress Bot commented Apr 14, 2026

Copy link
Copy Markdown

Activity    Run #3603

Run Properties:  status check passed Passed #3603  •  git commit a3f6253f6f: fix(notifications): Require absolute links for support of desktop and mobile cli...
Project Activity
Branch Review fix/sanitize-notification-urls
Run status status check passed Passed #3603
Run duration 02m 42s
Commit git commit a3f6253f6f: fix(notifications): Require absolute links for support of desktop and mobile cli...
Committer Anna
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 16
View all changes introduced in this branch ↗︎

@miaulalala
miaulalala force-pushed the fix/sanitize-notification-urls branch from a62486d to 11bc0d5 Compare April 14, 2026 20:47
Comment thread lib/NotificationGenerator.php Outdated
… mobile clients

To align with the stricter validation introduced in nextcloud/server#59606.

AI-Assisted-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Anna Larch <anna@nextcloud.com>
@Altahrim Altahrim added this to the Nextcloud 34.0.3 milestone Jul 27, 2026
@miaulalala
miaulalala merged commit b793bd2 into master Jul 27, 2026
60 of 66 checks passed
@miaulalala
miaulalala deleted the fix/sanitize-notification-urls branch July 27, 2026 17:56
@miaulalala

Copy link
Copy Markdown
Collaborator Author

/backport to stable34

Sign up for free to 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.

6 participants