benoitc/gunicorn

sendfile example does not use sendfile

Chiusa

#2279 aperta il 27 feb 2020

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (1 assegnatario)Python (1707 fork)batch import
To Schedulegood first issue

Metriche repository

Star
 (9334 stelle)
Metriche merge PR
 (Merge medio 8h 8m) (1 PR mergiata in 30 g)

Descrizione

The sendfile example application does not actually result in a call to sendfile. I verified this by adding code to http/wsgi.py and also looking at the output of strace.

It works if I remove the @validator decorator on the app method in sendfile.py.

I guess this is because the worker is checking for an instance of the write type, but the type is not the same with the validator wrapper. workers/sync.py handle_request() says

if isinstance(respiter, environ['wsgi.file_wrapper']):

Guida contributor

Direzione di ricerca
Examine the type check in workers/sync.py handle request, specifically the line 'if isinstance(respiter, environ["wsgi.file wrapper"])'. The issue suggests that a decorator changes the type, so the fix may involve stripping the wrapper or adjusting the type check. Review the sendfile.py example and consider using functools.wraps to preserve type information. Test with strace to confirm sendfile syscall is invoked.
Tech stack
python
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
understanding of Python decoratorsbasic WSGI knowledge
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60