Bug in value2Interface(): reflect.Bytes() called on Array value

[teejays]

I noticed that in value2Interface function in session_convert.go, we are using reflect.Value.Bytes() function when the fieldType is reflect.Array OR reflect.Slice. However, go's reflect.Value.Bytes() only works when the value is of type reflect.Slice, and panics otherwise.

I would propose that change the if condition from:

if (k == reflect.Array || k == reflect.Slice) && (...) {
    // logic
}

to

if (k == reflect.Slice) && (...) {
    // logic
}

[lunny]

@teejays could you send a PR to fix that? Thanks.

[teejays]

Thanks @lunny, I created a PR: #1346

[lunny]

Closed by #1346