Details
-
Bug
-
Resolution: Fixed
-
Major
-
None
-
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
- is cloned by
-
GOCBC-1115 [2.2] - View error contents parsed incorrectly
- Closed
For Gerrit Dashboard: GOCBC-1095 | ||||||
---|---|---|---|---|---|---|
# | Subject | Branch | Project | Status | CR | V |
153253,3 | GOCBC-1095: Check for nil error when parsing view errors | master | gocbcore | Status: MERGED | +2 | +1 |
155482,2 | GOCBC-1095: Check for nil error when parsing view errors | v9.1 | gocbcore | Status: MERGED | +2 | +1 |
159219,2 | Upgrade gocbcore to 9.1.5+ | cheshire-cat | cbft | Status: MERGED | +2 | +1 |
159221,2 | Upgrade gocbcore to 9.1.5+ | cheshire-cat | n1fty | Status: MERGED | +2 | +1 |
159222,2 | Upgrade gocbcore to 9.1.5+ | cheshire-cat | query | Status: MERGED | +2 | +1 |
159232,2 | Upgrade gocbcore to 9.1.5+ | master | n1fty | Status: MERGED | +2 | +1 |
159233,2 | Upgrade gocbcore to 9.1.5+ | master | query | Status: MERGED | +2 | +1 |