2015-06-01 20:55:27 +03:00
|
|
|
package server
|
|
|
|
|
|
|
|
import (
|
|
|
|
"bytes"
|
2015-11-25 22:50:05 +03:00
|
|
|
|
2015-11-28 14:22:29 +03:00
|
|
|
"github.com/micro/go-micro/codec"
|
2019-01-07 16:48:38 +03:00
|
|
|
raw "github.com/micro/go-micro/codec/bytes"
|
2019-01-02 15:55:06 +03:00
|
|
|
"github.com/micro/go-micro/codec/grpc"
|
|
|
|
"github.com/micro/go-micro/codec/json"
|
2015-11-28 21:54:38 +03:00
|
|
|
"github.com/micro/go-micro/codec/jsonrpc"
|
2019-01-02 15:55:06 +03:00
|
|
|
"github.com/micro/go-micro/codec/proto"
|
2015-11-28 21:54:38 +03:00
|
|
|
"github.com/micro/go-micro/codec/protorpc"
|
2015-11-20 19:17:33 +03:00
|
|
|
"github.com/micro/go-micro/transport"
|
server/rpc_codec: if c.codec.Write fails, reset write buffer and encode an error message about the encoding failure
When developing go-micro services, it is frequently possible to set invalid results in the response pointer. When this happens (as I and @trushton personally experienced), `sendResponse()` returns an error correctly explaining what happened (e.g. protobuf refused to encode a bad struct) but the `call()` function one above it in the stack ignores the returned error object.
Thus, invalid structs go un-encoded and the _client side times out_. @trushton and I first caught this in our CI builds when we left a protobuf.Empty field uninitialized (nil) instead of setting it to `&ptypes.Empty{}`. This resulted in an `proto: oneof field has nil value` error, but it was dropped and became a terribly confusing client timeout instead.
This patch is two independent changes:
* In rpc_codec, when a serialization failure occurs serialize an error message, which will correctly become a 500 for HTTP services, about the encoding failure. This means rpc_codec only returns an `error` when a socket failure occurs, which I believe is the behavior that rpc_service is expecting anyway.
* In rpc_service, log any errors returned by sendResponse instead of dropping the error object. This will make debugging client timeouts less of a hassle.
2017-07-12 00:51:36 +03:00
|
|
|
"github.com/pkg/errors"
|
2015-06-01 20:55:27 +03:00
|
|
|
)
|
|
|
|
|
2018-11-23 23:05:31 +03:00
|
|
|
type rpcCodec struct {
|
2015-06-01 20:55:27 +03:00
|
|
|
socket transport.Socket
|
2015-11-28 14:22:29 +03:00
|
|
|
codec codec.Codec
|
2019-01-08 18:38:25 +03:00
|
|
|
first bool
|
2015-06-01 20:55:27 +03:00
|
|
|
|
|
|
|
req *transport.Message
|
2015-10-16 00:06:43 +03:00
|
|
|
buf *readWriteCloser
|
|
|
|
}
|
2015-06-01 20:55:27 +03:00
|
|
|
|
2015-10-16 00:06:43 +03:00
|
|
|
type readWriteCloser struct {
|
2015-06-01 20:55:27 +03:00
|
|
|
wbuf *bytes.Buffer
|
|
|
|
rbuf *bytes.Buffer
|
|
|
|
}
|
|
|
|
|
2015-11-28 14:22:29 +03:00
|
|
|
var (
|
2019-01-07 16:48:38 +03:00
|
|
|
DefaultContentType = "application/protobuf"
|
|
|
|
|
|
|
|
DefaultCodecs = map[string]codec.NewCodec{
|
2019-01-02 15:55:06 +03:00
|
|
|
"application/grpc": grpc.NewCodec,
|
|
|
|
"application/grpc+json": grpc.NewCodec,
|
|
|
|
"application/grpc+proto": grpc.NewCodec,
|
2019-01-01 01:01:16 +03:00
|
|
|
"application/json": json.NewCodec,
|
2015-11-28 21:54:38 +03:00
|
|
|
"application/json-rpc": jsonrpc.NewCodec,
|
2019-01-01 01:01:16 +03:00
|
|
|
"application/protobuf": proto.NewCodec,
|
2015-11-28 21:54:38 +03:00
|
|
|
"application/proto-rpc": protorpc.NewCodec,
|
2019-01-07 16:48:38 +03:00
|
|
|
"application/octet-stream": raw.NewCodec,
|
2015-11-28 14:22:29 +03:00
|
|
|
}
|
|
|
|
)
|
|
|
|
|
2015-10-16 00:06:43 +03:00
|
|
|
func (rwc *readWriteCloser) Read(p []byte) (n int, err error) {
|
|
|
|
return rwc.rbuf.Read(p)
|
2015-06-01 20:55:27 +03:00
|
|
|
}
|
|
|
|
|
2015-10-16 00:06:43 +03:00
|
|
|
func (rwc *readWriteCloser) Write(p []byte) (n int, err error) {
|
|
|
|
return rwc.wbuf.Write(p)
|
|
|
|
}
|
2015-06-01 20:55:27 +03:00
|
|
|
|
2015-10-16 00:06:43 +03:00
|
|
|
func (rwc *readWriteCloser) Close() error {
|
|
|
|
rwc.rbuf.Reset()
|
|
|
|
rwc.wbuf.Reset()
|
|
|
|
return nil
|
|
|
|
}
|
|
|
|
|
2019-01-08 18:38:25 +03:00
|
|
|
func newRpcCodec(req *transport.Message, socket transport.Socket, c codec.NewCodec) codec.Codec {
|
2015-11-25 22:50:05 +03:00
|
|
|
rwc := &readWriteCloser{
|
|
|
|
rbuf: bytes.NewBuffer(req.Body),
|
|
|
|
wbuf: bytes.NewBuffer(nil),
|
|
|
|
}
|
2018-11-23 23:05:31 +03:00
|
|
|
r := &rpcCodec{
|
2019-01-08 18:38:25 +03:00
|
|
|
first: true,
|
2015-11-25 22:50:05 +03:00
|
|
|
buf: rwc,
|
2015-11-28 14:22:29 +03:00
|
|
|
codec: c(rwc),
|
2015-10-16 13:15:17 +03:00
|
|
|
req: req,
|
2015-11-25 22:50:05 +03:00
|
|
|
socket: socket,
|
2015-06-01 20:55:27 +03:00
|
|
|
}
|
2015-10-16 00:06:43 +03:00
|
|
|
return r
|
|
|
|
}
|
|
|
|
|
2019-01-08 18:38:25 +03:00
|
|
|
func (c *rpcCodec) ReadHeader(r *codec.Message, t codec.MessageType) error {
|
2019-01-09 19:20:57 +03:00
|
|
|
// the initieal message
|
|
|
|
m := codec.Message{
|
|
|
|
Header: c.req.Header,
|
|
|
|
Body: c.req.Body,
|
|
|
|
}
|
2015-12-18 23:28:50 +03:00
|
|
|
|
2019-01-08 18:38:25 +03:00
|
|
|
// if its a follow on request read it
|
|
|
|
if !c.first {
|
2015-12-18 23:28:50 +03:00
|
|
|
var tm transport.Message
|
2019-01-08 18:38:25 +03:00
|
|
|
|
|
|
|
// read off the socket
|
2015-12-18 23:28:50 +03:00
|
|
|
if err := c.socket.Recv(&tm); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
2019-01-08 18:38:25 +03:00
|
|
|
// reset the read buffer
|
2015-12-18 23:28:50 +03:00
|
|
|
c.buf.rbuf.Reset()
|
2019-01-08 18:38:25 +03:00
|
|
|
|
|
|
|
// write the body to the buffer
|
2015-12-18 23:28:50 +03:00
|
|
|
if _, err := c.buf.rbuf.Write(tm.Body); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
|
|
|
|
2019-01-08 18:38:25 +03:00
|
|
|
// set the message header
|
2016-01-28 21:11:13 +03:00
|
|
|
m.Header = tm.Header
|
2019-01-09 19:20:57 +03:00
|
|
|
// set the message body
|
|
|
|
m.Body = tm.Body
|
2015-12-16 05:15:09 +03:00
|
|
|
}
|
|
|
|
|
2019-01-08 18:38:25 +03:00
|
|
|
// no longer first read
|
|
|
|
c.first = false
|
|
|
|
|
2019-01-01 01:01:16 +03:00
|
|
|
// set some internal things
|
|
|
|
m.Target = m.Header["X-Micro-Service"]
|
2019-01-11 00:25:31 +03:00
|
|
|
m.Endpoint = m.Header["X-Micro-Endpoint"]
|
2019-01-08 18:38:25 +03:00
|
|
|
m.Id = m.Header["X-Micro-Id"]
|
2019-01-07 12:11:36 +03:00
|
|
|
|
|
|
|
// read header via codec
|
2015-11-28 14:22:29 +03:00
|
|
|
err := c.codec.ReadHeader(&m, codec.Request)
|
2019-01-08 18:38:25 +03:00
|
|
|
|
|
|
|
// set the method/id
|
2019-01-11 00:25:31 +03:00
|
|
|
r.Endpoint = m.Endpoint
|
2019-01-08 18:38:25 +03:00
|
|
|
r.Id = m.Id
|
2019-01-07 12:11:36 +03:00
|
|
|
|
2015-11-28 14:22:29 +03:00
|
|
|
return err
|
2015-06-01 20:55:27 +03:00
|
|
|
}
|
|
|
|
|
2019-01-07 21:20:47 +03:00
|
|
|
func (c *rpcCodec) ReadBody(b interface{}) error {
|
2015-11-28 14:22:29 +03:00
|
|
|
return c.codec.ReadBody(b)
|
2015-06-01 20:55:27 +03:00
|
|
|
}
|
|
|
|
|
2019-01-09 19:20:57 +03:00
|
|
|
func (c *rpcCodec) Write(r *codec.Message, b interface{}) error {
|
2015-10-16 00:06:43 +03:00
|
|
|
c.buf.wbuf.Reset()
|
2019-01-08 18:38:25 +03:00
|
|
|
|
|
|
|
// create a new message
|
2015-11-28 14:22:29 +03:00
|
|
|
m := &codec.Message{
|
2019-01-11 00:25:31 +03:00
|
|
|
Endpoint: r.Endpoint,
|
|
|
|
Id: r.Id,
|
|
|
|
Error: r.Error,
|
|
|
|
Type: r.Type,
|
2019-01-11 16:44:47 +03:00
|
|
|
Header: map[string]string{},
|
|
|
|
}
|
|
|
|
|
|
|
|
// set request id
|
|
|
|
if len(r.Id) > 0 {
|
|
|
|
m.Header["X-Micro-Id"] = r.Id
|
|
|
|
}
|
|
|
|
|
|
|
|
// set target
|
|
|
|
if len(r.Target) > 0 {
|
|
|
|
m.Header["X-Micro-Service"] = r.Target
|
|
|
|
}
|
|
|
|
|
|
|
|
// set request endpoint
|
|
|
|
if len(r.Endpoint) > 0 {
|
|
|
|
m.Header["X-Micro-Endpoint"] = r.Endpoint
|
|
|
|
}
|
|
|
|
|
|
|
|
if len(r.Error) > 0 {
|
|
|
|
m.Header["X-Micro-Error"] = r.Error
|
2015-11-28 14:22:29 +03:00
|
|
|
}
|
2019-01-08 18:38:25 +03:00
|
|
|
|
2019-01-09 19:20:57 +03:00
|
|
|
// the body being sent
|
|
|
|
var body []byte
|
|
|
|
|
|
|
|
// if we have encoded data just send it
|
|
|
|
if len(r.Body) > 0 {
|
|
|
|
body = r.Body
|
|
|
|
// write to the body
|
|
|
|
} else if err := c.codec.Write(m, b); err != nil {
|
server/rpc_codec: if c.codec.Write fails, reset write buffer and encode an error message about the encoding failure
When developing go-micro services, it is frequently possible to set invalid results in the response pointer. When this happens (as I and @trushton personally experienced), `sendResponse()` returns an error correctly explaining what happened (e.g. protobuf refused to encode a bad struct) but the `call()` function one above it in the stack ignores the returned error object.
Thus, invalid structs go un-encoded and the _client side times out_. @trushton and I first caught this in our CI builds when we left a protobuf.Empty field uninitialized (nil) instead of setting it to `&ptypes.Empty{}`. This resulted in an `proto: oneof field has nil value` error, but it was dropped and became a terribly confusing client timeout instead.
This patch is two independent changes:
* In rpc_codec, when a serialization failure occurs serialize an error message, which will correctly become a 500 for HTTP services, about the encoding failure. This means rpc_codec only returns an `error` when a socket failure occurs, which I believe is the behavior that rpc_service is expecting anyway.
* In rpc_service, log any errors returned by sendResponse instead of dropping the error object. This will make debugging client timeouts less of a hassle.
2017-07-12 00:51:36 +03:00
|
|
|
c.buf.wbuf.Reset()
|
2019-01-08 18:38:25 +03:00
|
|
|
|
|
|
|
// write an error if it failed
|
server/rpc_codec: if c.codec.Write fails, reset write buffer and encode an error message about the encoding failure
When developing go-micro services, it is frequently possible to set invalid results in the response pointer. When this happens (as I and @trushton personally experienced), `sendResponse()` returns an error correctly explaining what happened (e.g. protobuf refused to encode a bad struct) but the `call()` function one above it in the stack ignores the returned error object.
Thus, invalid structs go un-encoded and the _client side times out_. @trushton and I first caught this in our CI builds when we left a protobuf.Empty field uninitialized (nil) instead of setting it to `&ptypes.Empty{}`. This resulted in an `proto: oneof field has nil value` error, but it was dropped and became a terribly confusing client timeout instead.
This patch is two independent changes:
* In rpc_codec, when a serialization failure occurs serialize an error message, which will correctly become a 500 for HTTP services, about the encoding failure. This means rpc_codec only returns an `error` when a socket failure occurs, which I believe is the behavior that rpc_service is expecting anyway.
* In rpc_service, log any errors returned by sendResponse instead of dropping the error object. This will make debugging client timeouts less of a hassle.
2017-07-12 00:51:36 +03:00
|
|
|
m.Error = errors.Wrapf(err, "Unable to encode body").Error()
|
2019-01-07 12:11:36 +03:00
|
|
|
m.Header["X-Micro-Error"] = m.Error
|
2019-01-09 19:20:57 +03:00
|
|
|
// no body to write
|
server/rpc_codec: if c.codec.Write fails, reset write buffer and encode an error message about the encoding failure
When developing go-micro services, it is frequently possible to set invalid results in the response pointer. When this happens (as I and @trushton personally experienced), `sendResponse()` returns an error correctly explaining what happened (e.g. protobuf refused to encode a bad struct) but the `call()` function one above it in the stack ignores the returned error object.
Thus, invalid structs go un-encoded and the _client side times out_. @trushton and I first caught this in our CI builds when we left a protobuf.Empty field uninitialized (nil) instead of setting it to `&ptypes.Empty{}`. This resulted in an `proto: oneof field has nil value` error, but it was dropped and became a terribly confusing client timeout instead.
This patch is two independent changes:
* In rpc_codec, when a serialization failure occurs serialize an error message, which will correctly become a 500 for HTTP services, about the encoding failure. This means rpc_codec only returns an `error` when a socket failure occurs, which I believe is the behavior that rpc_service is expecting anyway.
* In rpc_service, log any errors returned by sendResponse instead of dropping the error object. This will make debugging client timeouts less of a hassle.
2017-07-12 00:51:36 +03:00
|
|
|
if err := c.codec.Write(m, nil); err != nil {
|
|
|
|
return err
|
|
|
|
}
|
2019-01-09 19:20:57 +03:00
|
|
|
// write the body
|
|
|
|
} else {
|
|
|
|
// set the body
|
|
|
|
body = c.buf.wbuf.Bytes()
|
2015-06-01 20:55:27 +03:00
|
|
|
}
|
2015-12-16 05:15:09 +03:00
|
|
|
|
2019-01-11 16:44:47 +03:00
|
|
|
// Set content type if theres content
|
|
|
|
if len(body) > 0 {
|
|
|
|
m.Header["Content-Type"] = c.req.Header["Content-Type"]
|
|
|
|
}
|
|
|
|
|
2019-01-08 18:38:25 +03:00
|
|
|
// send on the socket
|
2015-06-01 20:55:27 +03:00
|
|
|
return c.socket.Send(&transport.Message{
|
2016-01-28 21:11:13 +03:00
|
|
|
Header: m.Header,
|
2019-01-09 19:20:57 +03:00
|
|
|
Body: body,
|
2015-06-01 20:55:27 +03:00
|
|
|
})
|
|
|
|
}
|
|
|
|
|
2018-11-23 23:05:31 +03:00
|
|
|
func (c *rpcCodec) Close() error {
|
2015-10-16 00:06:43 +03:00
|
|
|
c.buf.Close()
|
2015-11-28 14:22:29 +03:00
|
|
|
c.codec.Close()
|
2015-06-01 20:55:27 +03:00
|
|
|
return c.socket.Close()
|
|
|
|
}
|
2019-01-08 18:38:25 +03:00
|
|
|
|
|
|
|
func (c *rpcCodec) String() string {
|
|
|
|
return "rpc"
|
|
|
|
}
|