Skip to content

fix: propfind query monitor breaking removeListener and removeAllList… - #54507

Merged
CarlSchwan merged 1 commit into
masterfrom
fix/54501/removeListener
Sep 3, 2025
Merged

fix: propfind query monitor breaking removeListener and removeAllList…#54507
CarlSchwan merged 1 commit into
masterfrom
fix/54501/removeListener

Conversation

@salmart-dev

@salmart-devsalmart-dev commented Aug 19, 2025

Copy link
Copy Markdown
Contributor

Summary

This PR fixes the issue that when debug mode is enabled, removeListener and removeAllListeners stop working on Connector\Sabre\Server instances.

Checklist

@salmart-devsalmart-dev self-assigned this Aug 19, 2025
@salmart-devsalmart-dev added the 3. to review Waiting for reviews label Aug 19, 2025
@salmart-dev
salmart-devforce-pushed the fix/54501/removeListener branch from 1a664f2 to d281a97CompareAugust 19, 2025 09:28
@salmart-dev
salmart-dev marked this pull request as ready for review August 19, 2025 09:28
@salmart-dev
salmart-dev requested a review from a team as a code ownerAugust 19, 2025 09:28
@salmart-dev
salmart-dev requested review from Altahrim, CarlSchwan and leftybournes and removed request for a teamAugust 19, 2025 09:28
…eners
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>

@SebastianKrupinskiSebastianKrupinski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Stumbled on this looking to solve and issue with calendar, failing to create or delete events.

Tested, this seems to resolve the issue.

@CarlSchwan
CarlSchwan merged commit f60e7b9 into masterSep 3, 2025
205 of 207 checks passed
@CarlSchwan
CarlSchwan deleted the fix/54501/removeListener branch September 3, 2025 16:51
@nextcloud-botnextcloud-bot mentioned this pull request Sep 4, 2025
@blizzzblizzz mentioned this pull request Oct 14, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

After #54153, removeListener on Sabre DAV server doesn't work

3 participants

@salmart-dev@CarlSchwan@SebastianKrupinski