benoitc/gunicorn

sendfile example does not use sendfile

Geschlossen

#2.279 geöffnet am 27.02.2020

Auf GitHub ansehenContributor Guide
 (4 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Python (1.707 Forks)batch import
To Schedulegood first issue

Repository-Metriken

Stars
 (9.334 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8h 8m) (1 gemergte PR in 30 T)

Beschreibung

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']):

Contributor Guide

Research-Richtung
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
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
understanding of Python decoratorsbasic WSGI knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60