failover URL should allow for NIO option

stompgem / stomp

A ruby gem for sending and receiving messages from a Stomp protocol compliant message queue. Includes: failover logic, ssl support.

http://stomp.github.com

Apache License 2.0

152 stars 80 forks source link

Closed PaulGale closed 11 years ago

PaulGale commented 11 years ago

The failover URL parsing logic in client/utils.rb line 41 should be:

/^failover:(\/\/)?\(stomp(\+(ssl|nio))?:\/\/#{url_regex}(,stomp(\+(ssl|nio))?:\/\/#{url_regex}\))+(\?(.*))?$/

Note the capture group numberings change as a result.

gmallard commented 11 years ago

What gem behavior would change by using a stomp+nio// URL?

PaulGale commented 11 years ago

None I suppose. Just some added clarity. No big deal.

gmallard commented 11 years ago

Closed per discussion.