Skip to content

fix: truncating caldav subscription data before saving on DB - #63295

Open
madbob wants to merge 1 commit into
nextcloud:masterfrom
madbob:issue_54859
Open

fix: truncating caldav subscription data before saving on DB#63295
madbob wants to merge 1 commit into
nextcloud:masterfrom
madbob:issue_54859

Conversation

@madbob

@madbob madbob commented Aug 15, 2026

Copy link
Copy Markdown

Summary

In $subscriptionPropertyMap I've added, for "string" properties, the maximum length, to know how to properly truncate incoming values.

Note: I've found that in $subscriptionPropertyMap array both stripalarms and stripattachments were marked as "string", since the introduction of those datatypes. But in database schema those are always been "smallint" (boolean) values.

Checklist

@solracsf solracsf added 3. to review Waiting for reviews feature: caldav Related to CalDAV internals labels Aug 15, 2026
@solracsf solracsf added this to the Nextcloud 35 milestone Aug 15, 2026
Signed-off-by: Roberto Guido <info@madbob.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews feature: caldav Related to CalDAV internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Truncate calendar subscriptions display name to 100 chars before saving it

2 participants