search

Workable / flask-log-request-id

Flask extension to track and log Request-ID headers produced by PaaS like Heroku and load balancers like Amazon ELB
MIT License
113 stars 23 forks source link

Doesn't work with celery send_task or named signatures #31

Closed geniass closed 6 years ago

geniass commented 7 years ago

For my use case, I need to call celery tasks by name rather than importing the actual task object.

It seems like this never calls the apply_async method, so the request ID is never set in the headers.

I believe a better way to set the headers is using Celery's signal system. Specifically the before_task_publish hook.

I will open a pull request shortly.

sque commented 6 years ago

I think this was addressed by #34