Skip to content

feat(dav): dispatch SabrePluginAddEvent for the legacy CalDAV and CardDAV endpoints - #65816

Draft
karlitschek wants to merge 1 commit into
masterfrom
feat/legacy-dav-plugin-event
Draft

karlitschek wants to merge 1 commit into
masterfrom
feat/legacy-dav-plugin-event

Conversation

@karlitschek

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

A small core change used by the intrusion_detection app, alongside #65735–#65741. It can be reviewed and merged on its own: remote.php/caldav and remote.php/carddav now dispatch SabrePluginAddEvent, like remote.php/dav and remote.php/webdav.

TODO

  • Description by the author
  • Manual test on a live instance

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

AI disclosure: this change was written with Claude Code (claude-opus-5-5) and reviewed by the author; every commit carries an Assisted-by trailer.

🤖 Generated with Claude Code

…dDAV endpoints

remote.php/caldav and remote.php/carddav built their Sabre servers
without letting apps add plugins, unlike remote.php/dav and
remote.php/webdav. Plugins that apps add to every DAV server, such as
session checks, were missing on these endpoints. Both now dispatch
SabrePluginAddEvent right before the server starts, as webdav.php does.

Assisted-by: ClaudeCode:claude-opus-5-5
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

This branch has not been deployed

No deployments
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.

2 participants