unistack-org/micro
3
0
Fork 3
Code Issues 14 Pull Requests 2 Actions Packages Projects Releases 242 Wiki Activity

Fix http broker to allow multiple nodes for subscription, total hack

Browse Source
This commit is contained in:
Asim 2015-12-27 01:17:53 +00:00
parent 3dd911fb33
commit d0b2b612f3
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
broker/http_broker.go
View File

@ -260,15 +260,18 @@ func (h *httpBroker) Publish(topic string, msg *Message, opts ...PublishOption)
func (h *httpBroker) Subscribe(topic string, handler Handler, opts ...SubscribeOption) (Subscriber, error) {
opt := newSubscribeOptions(opts...)
fmt.Println("subscribe to", topic)
// parse address for host, port
parts := strings.Split(h.Address(), ":")
host := strings.Join(parts[:len(parts)-1], ":")
port, _ := strconv.Atoi(parts[len(parts)-1])
id := uuid.NewUUID().String()
// register service
node := &registry.Node{
Id: h.id,
Id: topic + "." + h.id + "." + id,
Address: host,
Port: port,
}
@ -286,7 +289,7 @@ func (h *httpBroker) Subscribe(topic string, handler Handler, opts ...SubscribeO
subscriber := &httpSubscriber{
opts: opt,
id: uuid.NewUUID().String(),
id: id,
topic: topic,
ch: h.unsubscribe,
fn: handler,
@ -299,8 +302,8 @@ func (h *httpBroker) Subscribe(topic string, handler Handler, opts ...SubscribeO
h.Lock()
h.subscribers[topic] = append(h.subscribers[topic], subscriber)
fmt.Println(h.subscribers)
h.Unlock()
return subscriber, nil
}