\Sabre\CalDAV\Schedule\Plugin::beforeUnbind calls \Sabre\DAVACL\IACL::getOwner without a type check #1345
Comments
ChristophWurst
added a commit
to nextcloud/server
that referenced
this issue
Jun 2, 2021
Due to a bug in Sabre it's necessary for calendar objects to implement ACLs. Otherwise the scheduling plugin will throw an error when it tries to fetch the owner of a calendar object that is being deleted. Ref sabre-io/dav#1345 Signed-off-by: Christoph Wurst <[email protected]>
ChristophWurst
added a commit
to nextcloud/server
that referenced
this issue
Jun 2, 2021
Due to a bug in Sabre it's necessary for calendar objects to implement ACLs. Otherwise the scheduling plugin will throw an error when it tries to fetch the owner of a calendar object that is being deleted. Ref sabre-io/dav#1345 Signed-off-by: Christoph Wurst <[email protected]>
|
Hey @ChristophWurst 👋 Feel free to submit a PR to address this. THX a lot! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
dav/lib/CalDAV/Schedule/Plugin.php
Line 389 in c1afdc7
The node might not have ACLs implemented. In that case this calls a method that doesn't exist.
In my case it's an
\Sabre\CalDAV\ICalendarObjectthat gets deleted. Those often have ACLs, but not always. It might make sense to exit early when IACL is not implemented by the node.Moreover a
nullowner will lead to a type issue.The text was updated successfully, but these errors were encountered: