Uploaded image for project: 'Couchbase Lite'
  1. Couchbase Lite
  2. CBL-525

Upgrade from 1.x fails if any docID starts with "_"

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.6.1
    • 2.7.0
    • LiteCore
    • Security Level: Public
    • None
    • "CBL 2.6 Android"

    Description

      Via this forum thread — apparently it was somehow possible in 1.x to create a document whose ID started with "_", even though this is supposed to be illegal. Any database with such a doc will fail to open in 2.x because the upgrader will reject that docID.

      We have a test in the Upgrader class to skip the special case of docIDs that start with "design/" (which was legal in 1.x, for complicated reasons.) This should be modified to skip (not reject) any doc whose ID starts with "".

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          jens Jens Alfke created issue -
          jens Jens Alfke added a comment -

          We have a PR (from the developer) which is tiny and looks very safe.

          jens Jens Alfke added a comment - We have a PR (from the developer) which is tiny and looks very safe.
          daniel.petersen Daniel Petersen made changes -
          Field Original Value New Value
          Fix Version/s Mercury [ 16188 ]
          daniel.petersen Daniel Petersen made changes -
          Assignee The Lite [ cbgto ] Jens Alfke [ jens ]
          jens Jens Alfke added a comment -

          Merged the PR.

          jens Jens Alfke added a comment - Merged the PR.
          jens Jens Alfke made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          daniel.petersen Daniel Petersen made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

          People

            jens Jens Alfke
            jens Jens Alfke
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty