jack77213
/
qingcloud-sdk-go
1
0
Fork 0
Code Issues Pull Requests Releases 3 Wiki Activity

Merge pull request #97 from cygnushan/fix-UnpackHTTPRequest-panic 

Fix unpack http request panic
This commit is contained in:
Simon 2018-06-01 14:21:45 +08:00 committed by GitHub
parent aa89edef9b 694c715a64
commit 05aacd4b8c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
request/unpacker.go
View File

@ -81,8 +81,8 @@ func (u *Unpacker) parseError() error {
retCodeValue := u.output.Elem().FieldByName("RetCode")
messageValue := u.output.Elem().FieldByName("Message")
if retCodeValue.IsValid() && retCodeValue.Type().String() == "*int" &&
messageValue.IsValid() && messageValue.Type().String() == "*string" &&
if retCodeValue.Elem().IsValid() && retCodeValue.Type().String() == "*int" &&
messageValue.Elem().IsValid() && messageValue.Type().String() == "*string" &&
retCodeValue.Elem().Int() != 0 {
return &errors.QingCloudError{

1
request/unpacker_test.go
View File

@ -79,6 +79,7 @@ func TestUnpackerUnpackHTTPRequest(t *testing.T) {
RetCode *int `json:"ret_code" name:"ret_code"`
TotalCount *int `json:"total_count" name:"total_count"`
VolumeSet []*Volume `json:"volume_set" name:"volume_set"`
Message *string `json:"message" name:"message"`
}
httpResponse := &http.Response{Header: http.Header{}}