#199 run fedmsg consumers with a Flask app context
Merged by dcallagh. Opened by dcallagh.
dcallagh/greenwave fedmsg-consumer-app-context  into  master

Download 199.patch

This lets us refer to current_app.config throughout the code, instead of
awkwardly having this thing where a piece of code can use
current_app.config if it was called through a Flask handler but
otherwise has to pass around an explicit config object.

The 'greenwave_cache' setting from the fedmsg config is dropped, because
we can just re-use the existing 'CACHE' setting from the Greenwave
config instead.

I had to fix this for PR#184, where I wanted to start calling other bits of code that rely on current_app.config from inside the message handlers.

Nice, :+1:

Pull-Request has been merged by dcallagh

Metadata