tlsAnalyzer does not declare its queue

[jvehent]

This call should be preceded by a queue declaration:
https://github.com/mozilla/TLS-Observer/blob/master/src/tlsAnalyser/tlsAnalyser.go#L239
(returned errors should be checked as well).

[0xdiba]

The queue is automatically declared by the amqpmodule wrapper I have built.
Don't know if its a good way to go, just wanted to take out logic from the
analysers.

You are absolutely right about the error handling.

Thanks for checking out the code ;) .

On Wed, Apr 15, 2015 at 4:00 PM, Julien Vehent notifications@github.com
wrote:

This call should be preceded by a queue declaration:

https://github.com/mozilla/TLS-Observer/blob/master/src/tlsAnalyser/tlsAnalyser.go#L239
(returned errors should be checked as well).

—
Reply to this email directly or view it on GitHub
#25.

[jvehent]

Ha, so, I think if you just add the queue name to this call instead of "" it will work 🌴
https://github.com/mozilla/TLS-Observer/blob/master/src/modules/amqpmodule/amqpmodule.go#L128

[0xdiba]

Oooops....

On Thu, Apr 16, 2015 at 3:21 PM, Julien Vehent notifications@github.com
wrote:

Ha, so, I think if you just add the queue name to this call instead of ""
it will work [image: 🌴]

https://github.com/mozilla/TLS-Observer/blob/master/src/modules/amqpmodule/amqpmodule.go#L128

—
Reply to this email directly or view it on GitHub
#25 (comment).

[0xdiba]

Relates to issue #27