Skip to content

[stable34] fix(notifications): notify mentioned users about mentions by guests - #9403

Merged
mejo- merged 1 commit into
stable34from
backport/9392/stable34
Oct 15, 2026
Merged

mejo- merged 1 commit into
stable34from
backport/9392/stable34

Conversation

@backportbot

@backportbot backportbot Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Backport of #9392

Warning, This backport's changes differ from the original and might be incomplete 鈿狅笍

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot
backportbot Bot requested review from benjaminfrueh and mejo- October 15, 2026 12:09
@backportbot backportbot Bot added bug Something isn't working 3. to review labels Oct 15, 2026
@backportbot backportbot Bot added this to the Nextcloud 34.0.6 milestone Oct 15, 2026
@mejo-
mejo- force-pushed the backport/9392/stable34 branch from 678e83f to 2cd0396 Compare October 15, 2026 14:02
@mejo-
mejo- marked this pull request as ready for review October 15, 2026 14:03
@mejo-
mejo- force-pushed the backport/9392/stable34 branch from 2cd0396 to aefc4dc Compare October 15, 2026 14:49
Store the guest name as a separate subject parameter and render it as a
guest rich object labelled "(guest)", so it cannot be mistaken for an
instance user. Notifications without a valid source are marked as
processed, which removes broken rows from earlier mentions. Mentions are
now rate limited for anonymous requests as well.

Fixes #8743

Signed-off-by: Jonas <jonas@freesources.org>
Assisted-by: ClaudeCode:claude-opus-5-5
@mejo-
mejo- force-pushed the backport/9392/stable34 branch from aefc4dc to de48401 Compare October 15, 2026 16:12
@mejo-
mejo- merged commit 37c0710 into stable34 Oct 15, 2026
80 of 83 checks passed
@mejo-
mejo- deleted the backport/9392/stable34 branch October 15, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant