New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gunicorn 19.0 broke the tornado worker #783

Closed
g-p-g opened this Issue Jun 13, 2014 · 6 comments

Comments

Projects
None yet
3 participants
@g-p-g

g-p-g commented Jun 13, 2014

After upgrading to gunicorn 19.0 the tornado worker which used to work with gunicorn 18.0 no longer does. It looks like the x_forwarded_for_header setting is no longer present in cfg (gunicorn.workers.gtornado.TornadoWorker @ run ~ line 95 or so), what's the expected way to use the new version?

[ERROR] Exception in worker process:
Traceback (most recent call last):
  File ".../python2.7/site-packages/gunicorn/arbiter.py", line 502, in spawn_worker
    worker.init_process()
  File ".../python2.7/site-packages/gunicorn/workers/base.py", line 120, in init_process
    self.run()
  File ".../python2.7/site-packages/gunicorn/workers/gtornado.py", line 96, in run
    server.xheaders = bool(self.cfg.x_forwarded_for_header)
  File ".../python2.7/site-packages/gunicorn/config.py", line 55, in __getattr__
    raise AttributeError("No configuration setting for: %s" % name)
AttributeError: No configuration setting for: x_forwarded_for_header
@tilgovi

This comment has been minimized.

Show comment
Hide comment
@tilgovi

tilgovi Jun 14, 2014

Collaborator

@benoitc looks like we broke tornado for real. We should do a quick fix release.

Collaborator

tilgovi commented Jun 14, 2014

@benoitc looks like we broke tornado for real. We should do a quick fix release.

@benoitc benoitc closed this in 7b902a2 Jun 14, 2014

@benoitc

This comment has been minimized.

Show comment
Hide comment
@benoitc

benoitc Jun 14, 2014

Owner

@tilgovi fixed. I will make a release today .

@g-p-g thanks for the report

Owner

benoitc commented Jun 14, 2014

@tilgovi fixed. I will make a release today .

@g-p-g thanks for the report

@g-p-g

This comment has been minimized.

Show comment
Hide comment
@g-p-g

g-p-g Jun 14, 2014

@benoitc that fix looks strange for an outsider as the removed line used to work before. Could you possibly point to some commit(s) that modified that behavior so it's no longer necessary to deal with X-FORWARDED-FOR in the tornado worker? Thanks.

g-p-g commented Jun 14, 2014

@benoitc that fix looks strange for an outsider as the removed line used to work before. Could you possibly point to some commit(s) that modified that behavior so it's no longer necessary to deal with X-FORWARDED-FOR in the tornado worker? Thanks.

@benoitc

This comment has been minimized.

Show comment
Hide comment
@benoitc

benoitc Jun 14, 2014

Owner

@g-p-g c487368

My bad I forgot to document it in the changelog. will do in the next one.

Owner

benoitc commented Jun 14, 2014

@g-p-g c487368

My bad I forgot to document it in the changelog. will do in the next one.

@benoitc

This comment has been minimized.

Show comment
Hide comment
@benoitc

benoitc Jun 14, 2014

Owner

@g-p-g also you can find the discussion in #633

Owner

benoitc commented Jun 14, 2014

@g-p-g also you can find the discussion in #633

@g-p-g

This comment has been minimized.

Show comment
Hide comment
@g-p-g

g-p-g Jun 14, 2014

@benoitc thanks for the attention, both are helpful.

g-p-g commented Jun 14, 2014

@benoitc thanks for the attention, both are helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment