mirror of
https://github.com/hanzoai/psrpc.git
synced 2026-09-30 23:47:22 +00:00
revert change to queue methods (#87)
* revert change to queue methods * claim channel is never queue * test
This commit is contained in:
@@ -49,7 +49,10 @@ func deserializeChannel(b []byte) (string, error) {
|
||||
|
||||
func deserialize(b []byte) (proto.Message, error) {
|
||||
a := &anypb.Any{}
|
||||
err := proto.Unmarshal(b, a)
|
||||
opt := proto.UnmarshalOptions{
|
||||
DiscardUnknown: true,
|
||||
}
|
||||
err := opt.Unmarshal(b, a)
|
||||
if err != nil {
|
||||
return nil, err
|
||||
}
|
||||
|
||||
@@ -69,7 +69,7 @@ func (i *RequestInfo) GetHandlerKey() string {
|
||||
func (i *RequestInfo) GetClaimResponseChannel() bus.Channel {
|
||||
return bus.Channel{
|
||||
Legacy: formatChannel('|', i.Service, i.Method, i.Topic, "RCLAIM"),
|
||||
Server: formatServerChannel(i.Service, i.Topic, i.Queue),
|
||||
Server: formatServerChannel(i.Service, i.Topic, false),
|
||||
Local: formatLocalChannel(i.Method, "RCLAIM"),
|
||||
}
|
||||
}
|
||||
|
||||
@@ -67,7 +67,7 @@ func TestChannelFormatters(t *testing.T) {
|
||||
require.Equal(t, "SRV.foo.a.b.c.Q", i.GetRPCChannel().Server)
|
||||
require.Equal(t, "bar.REQ", i.GetRPCChannel().Local)
|
||||
require.Equal(t, "foo|bar|a|b|c|RCLAIM", i.GetClaimResponseChannel().Legacy)
|
||||
require.Equal(t, "SRV.foo.a.b.c.Q", i.GetClaimResponseChannel().Server)
|
||||
require.Equal(t, "SRV.foo.a.b.c", i.GetClaimResponseChannel().Server)
|
||||
require.Equal(t, "bar.RCLAIM", i.GetClaimResponseChannel().Local)
|
||||
require.Equal(t, "foo|bar|a|b|c|STR", i.GetStreamServerChannel().Legacy)
|
||||
require.Equal(t, "SRV.foo.a.b.c", i.GetStreamServerChannel().Server)
|
||||
|
||||
+2
-3
@@ -44,9 +44,8 @@ func (s *ServiceDefinition) RegisterMethod(name string, affinityEnabled, multi,
|
||||
s.Methods.Store(name, &MethodInfo{
|
||||
AffinityEnabled: affinityEnabled,
|
||||
Multi: multi,
|
||||
// TODO: move queue check to generator
|
||||
RequireClaim: requireClaim && !queue,
|
||||
Queue: queue,
|
||||
RequireClaim: requireClaim,
|
||||
Queue: queue,
|
||||
})
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user