Requests: Die Schlüsselwort-Header müssen in der neuen Version 2.10.0 eine Zeichenfolge oder ein Puffer sein?

Erstellt am 5. Juli 2016  ·  7Kommentare  ·  Quelle: psf/requests

Der headers Schlüsselwörter
Wenn die Schlüsselwörter dnt und upgrade-insecure-requests Wert string oder buffer sind , ist es richtig, aber der Wert ist int, es ist falsch.Warum ist das so?
Rechts:

   'dnt': '1',
    'upgrade-insecure-requests': '1',

Falsch:

   'dnt': 1,
    'upgrade-insecure-requests': 1,

Wieso den?

Hilfreichster Kommentar

Unabhängig davon, was wir dokumentiert haben, hat dies früher funktioniert und jetzt nicht mehr. Wir werden eine Menge Code brechen, wenn wir dies nicht zurück ändern.

Alle 7 Kommentare

@wut0n9 Diese Verhaltensänderung ist nicht in v2.10.0, sondern im aktuellen Master-Zweig.

Das ist jedoch ein echter Fehler: #3366 hat dies rückgängig gemacht. @nateprewitt , sind Sie daran interessiert, dies mit einem Fix zu aktualisieren?

Header sollten immer Strings sein. Ich glaube nicht, dass dies eine signifikante Regression ist, wenn überhaupt. Wir haben immer dokumentiert, dass Headerwerte Zeichenfolgen sein sollten.

Unabhängig davon, was wir dokumentiert haben, hat dies früher funktioniert und jetzt nicht mehr. Wir werden eine Menge Code brechen, wenn wir dies nicht zurück ändern.

Ja, ich komme gleich darauf @Lukasa.

Siehe auch https://github.com/kennethreitz/requests/issues/865 und https://github.com/kennethreitz/requests/pull/866

Ich weiß nicht, wie viel "viel" in diesem Fall eigentlich ist.

@sigmavirus24 hat auf die relevanten früheren Meinungen verlinkt, die darauf hindeuten, dass wir eigentlich keine Nicht-String-Header-Werte zulassen. Das bedeutet also, dass dies kein Fehler ist: Wir haben eine Überarbeitung vorgenommen, die im Rahmen der API-Definition liegt.

@Lukasa richtig, ich bin ehrlich gesagt überrascht, dass das nicht früher

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen