search

tx7do / kratos-transport

kratos transport layer extension
MIT License
348 stars 95 forks source link

pulsar 的可以支持 kratos 2.7 吗？ #58

Closed moecasts closed 1 year ago

moecasts commented 1 year ago

没有实现 add 方法。

# github.com/tx7do/kratos-transport/transport/pulsar
/Users/caster/go/pkg/mod/github.com/tx7do/kratos-transport/transport/pulsar@v0.0.0-20230901150501-07ef396877dd/transport.go:40:9: cannot use tr.reqHeader (variable of type headerCarrier) as transport.Header value in return statement: headerCarrier does not implement transport.Header (missing method Add)
/Users/caster/go/pkg/mod/github.com/tx7do/kratos-transport/transport/pulsar@v0.0.0-20230901150501-07ef396877dd/transport.go:45:9: cannot use tr.replyHeader (variable of type headerCarrier) as transport.Header value in return statement: headerCarrier does not implement transport.Header (missing method Add)

相关 issue：https://github.com/tx7do/kratos-transport/issues/45

tx7do commented 1 year ago

更新到最新版。

moecasts commented 1 year ago

更新到最新版。

@tx7do 已经是最新版了，pulsar 的是没有实现的。 https://github.com/tx7do/kratos-transport/blob/main/transport/pulsar/transport.go

tx7do commented 1 year ago

确实，已经fix了。

moecasts commented 1 year ago

好的，谢谢大佬～