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
+407 −1

Commits on Jul 2, 2023

  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 <tcit@tcit.fr>
    @tcitworld
    tcitworld committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    2340534 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 <tcit@tcit.fr>
    @tcitworld
    tcitworld committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    3530ea8 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 <tcit@tcit.fr>
    @tcitworld
    tcitworld committed Jul 2, 2023
    Configuration menu
    Copy the full SHA
    b4118da View commit details
    Browse the repository at this point in the history