Produits Entr'ouvert » Passerelle

1. List of LoggedRequest Resoone Content-Type to log

Resoone ?

if re.match(r'%s' % content_type, ctype):

Pourquoi r'%s' % content_type et pas juste content_type ?

(changements d'indentation gratuits)

:/ Il y a juste la fin qui change, right ?

(à noter aussi que le contenu était de toute façon derrière un %r, donc il n'y avait pas vraiment de problème d'encodage dans #14191)

Aussi à pointer, dans les tests, un truc comme content='<FF><D8><FF><E0>^^PJFIF^A^A^A',, c'est de la chaine tout à fait imprimable; si tu veux faire un truc qui ressemble vraiment à du binaire de fichier jpeg, ça ressemblerait plutôt à '\xff\xd8\xff\xe0\x00\x10JFIF\x00\x01\x01\x01\x00'. (pareil côté pdf)

(mais comme l'intérêt n'est pas vraiment dans le contenu, autant peut-être taper un truc genre content='binary content, to be ignored'.

:/ Il y a juste la fin qui change, right ?

Comme je ne parviens pas à lire le patch et que je n'ai pas de réponse, je ne relirai pas.

-            self.logger.info('Request Payload: %r' %(response.request.body),
-                extra={'requests_request_payload': '%r' %response.request.body})
-        self.logger.info('Status code: %r' %(response.status_code),
-            extra={'requests_response_status': response.status_code})
+            self.logger.info('Request Payload: %r' % (response.request.body),
+                             extra={'requests_request_payload': '%r' % response.request.body})
+        self.logger.info('Status code: %r' % (response.status_code),
+                         extra={'requests_response_status': response.status_code})

Serghei Mihai a écrit :

[...]
ici tu re-indentes juste extra ?

Yep

ici tu re-indentes juste extra ?

Yep

Pour être on ne peut plus clair; tu peux éliminer de ce patch les réindentations ?

Tu n'as pas

============================================================================================= ERRORS =============================================================================================
_________________________________________________________________________ ERROR collecting tests/test_loggedrequests.py __________________________________________________________________________
tests/test_loggedrequests.py:9: in <module>
    from .utils import FakedResponse
E   ValueError: Attempted relative import in non-package

Quand tu lances les tests sur ta machine ?

Ok