Uploaded image for project: 'Couchbase Go SDK'
  1. Couchbase Go SDK
  2. GOCBC-1095

View error contents parsed incorrectly

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • core-10.0.0, core-9.1.5
    • None
    • None
    • 1

    Description

      		if err := json.Unmarshal(respBody, &errsArr); err != nil {
      			errorDescs = make([]ViewQueryErrorDesc, len(errsArr))
      			for errIdx, errMessage := range errsArr {
      				errorDescs[errIdx] = ViewQueryErrorDesc{
      					SourceNode: "",
      					Message:    errMessage,
      				}
      			}
      		} else if err := json.Unmarshal(respBody, &errsMap); err != nil {
      			for errNode, errMessage := range errsMap {
      				errorDescs = append(errorDescs, ViewQueryErrorDesc{
      					SourceNode: errNode,
      					Message:    errMessage,
      				})
      			}
      		}
      

      These checks should both be `err == nil`

      Attachments

        Issue Links

          For Gerrit Dashboard: GOCBC-1095
          # Subject Branch Project Status CR V

          Activity

            charles.dixon Charles Dixon created issue -
            charles.dixon Charles Dixon made changes -
            Field Original Value New Value
            Status New [ 10003 ] Open [ 1 ]
            charles.dixon Charles Dixon made changes -
            Status Open [ 1 ] In Progress [ 3 ]
            charles.dixon Charles Dixon made changes -
            Resolution Fixed [ 1 ]
            Status In Progress [ 3 ] Resolved [ 5 ]
            charles.dixon Charles Dixon made changes -
            Link This issue is cloned by GOCBC-1115 [ GOCBC-1115 ]
            charles.dixon Charles Dixon made changes -
            Fix Version/s core-9.1.5 [ 17641 ]
            Fix Version/s 2.3.0 [ 17235 ]

            People

              charles.dixon Charles Dixon
              charles.dixon Charles Dixon
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty