Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix dav properties column types #39084

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Fix dav properties column types #39084

wants to merge 3 commits into from
+322 −1

Commits on Oct 15, 2024

  1. fix(dav): change type of propertyvalue column to blob 

    The propertyvalue column can contain null 0x00 characters values because of serializing PHP objects since #30368. This truncates data in text fields, but not blob fields. We start by removing invalid value and altering the column to match the new type.

That's what Sabre PDO's being doing in the first place 🙈

Closes #37754

Signed-off-by: Thomas Citharel <[email protected]>
    @tcitworld
    tcitworld committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    e8d684d View commit details
    Browse the repository at this point in the history

  2. fix(dav): increase length of propertypath column 

    Match it to the value in oc_filecache table path column should be fine.

Closes #9907

Signed-off-by: Thomas Citharel <[email protected]>
    @tcitworld
    tcitworld committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    4c82fc4 View commit details
    Browse the repository at this point in the history

  3. chore(dav): bump dav app version for new migrations 

    Signed-off-by: Thomas Citharel <[email protected]>
    @tcitworld
    tcitworld committed Oct 15, 2024
    Configuration menu
    Copy the full SHA
    1440e5a View commit details
    Browse the repository at this point in the history