Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: 1.2.1
    • Component/s: library
    • Labels:
      None

      Description

      The error format is changing, need to ensure that's updated in the client.

      Rough example format:

      {"total_rows":0,"rows":[

      ],
      "errors": [

      {"from":"127.0.0.1:5984","reason":"Design document `_design/testfoobar` missing in database `test_db_b`."}

      ,

      {"from":"http://localhost:5984/_view_merge/","reason":"Design document `_design/testfoobar` missing in database `test_db_c`."}

      ]}

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

        Activity

        Hide
        ingenthr Matt Ingenthron added a comment -

        Here's a better example of the format. It changed slightly again.

        {"total_rows":0,"rows":[
        ],
        "errors":[

        {"from":"local","reason":"Design document `_design/testfoobar` missing in database `test_db_b`."}

        ,

        {"from":"http://localhost:5984/_view_merge/","reason":"Design document `_design/testfoobar` missing in database `test_db_c`."}

        ]}

        Show
        ingenthr Matt Ingenthron added a comment - Here's a better example of the format. It changed slightly again. {"total_rows":0,"rows":[ ], "errors":[ {"from":"local","reason":"Design document `_design/testfoobar` missing in database `test_db_b`."} , {"from":"http://localhost:5984/_view_merge/","reason":"Design document `_design/testfoobar` missing in database `test_db_c`."} ]}
        Hide
        ingenthr Matt Ingenthron added a comment -

        Was this handled in 0.9?

        Show
        ingenthr Matt Ingenthron added a comment - Was this handled in 0.9?
        Hide
        ingenthr Matt Ingenthron added a comment -

        The code needs to be inspected to verify this error handling.

        Show
        ingenthr Matt Ingenthron added a comment - The code needs to be inspected to verify this error handling.
        Hide
        john John Zablocki (Inactive) added a comment -

        Code is handling errors, but need unit test.

        Show
        john John Zablocki (Inactive) added a comment - Code is handling errors, but need unit test.
        Hide
        john John Zablocki (Inactive) added a comment -

        This error format is out of date. Current error format is below. As part of NCBC-165, error handling was cleaned up.

        {"error":"error","reason":"Error opening view `does_not_exist`, from set `default`, design document `_design/cities`:

        {not_found,\n missing_named_view}

        "}

        http://localhost:8092/default/_design/cities/_view/does_not_exist

        Show
        john John Zablocki (Inactive) added a comment - This error format is out of date. Current error format is below. As part of NCBC-165 , error handling was cleaned up. {"error":"error","reason":"Error opening view `does_not_exist`, from set `default`, design document `_design/cities`: {not_found,\n missing_named_view} "} http://localhost:8092/default/_design/cities/_view/does_not_exist

          People

          • Assignee:
            john John Zablocki (Inactive)
            Reporter:
            ingenthr Matt Ingenthron
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Gerrit Reviews

              There are no open Gerrit changes