jcelliott / turnpike

Go implementation of a WAMP (Web Application Messaging Protocol) client and router
MIT License
258 stars 88 forks source link

Options #97

Closed mourad closed 8 years ago

mourad commented 9 years ago

@jcelliott : please check if this looks ok and if where I added options on the interface makes sense.

Note that I reordered the arguments of Register as I thought it made sense to keep the MethodHandler as the last argument. Let me know if you agree, otherwise, I can change this back and reorder any of the functions.

Review on Reviewable

gngeorgiev commented 8 years ago

does the library actually support options? I tried with exclude_me but it seems to be ignored.

mourad commented 8 years ago

Closing this PR to avoid changes to the API .. for what I was attempting to accomplish, can simply use Send