PR#88: self.credentials is not a thing. - taskotron/resultsdb

taskotron / resultsdb

#88 self.credentials is not a thing.

Merged 6 years ago by jskladan. Opened 6 years ago by ralph.

taskotron/ ralph/resultsdb stomp-creds into develop

self.credentials is not a thing.

Ralph Bean • 6 years ago

50545c5

resultsdb/messaging.py

file modified

+1 -1

		`@@ -127,7 +127,7 @@`

		`conn = self.stomp.Connection(**self.connection)`
		`conn.start()`
		`- conn.connect(**self.credentials)`
		`+ conn.connect()`
		`try:`
		`conn.send(**kwargs)`
		`finally:`

ralph commented 6 years ago

I'm not sure how this made it in last time, but self.credentials is not
defined, nor is it needed. All of the connection information should be stored
in self.connection.

Found via testing against a real live ActiveMQ broker.