python-hyper / python-hyper/rfc3986

uri_reference.is_valid() accepts invalid hostnames and out-of-range ports

Offen
#138 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Python
Sterne
189
Forks
38
Ø Merge
13 Std. 36 Min.
Gemergte PRs (30 T.)
1

Beschreibung

The validator returns True for inputs that violate URI rules.

Examples:

In [15]:  uri_reference('google.com/path').is_valid()
Out[15]: True
In [27]:  uri_reference('http://example.com:99999/').is_valid()
Out[27]: True

Expected behavior:

The validator should reject missing schemes when a hostname is present
and should reject port values greater than 65535.

Actual behavior:

is_valid() returns True for both invalid inputs.

Steps to reproduce:

  1. Run the two example expressions in a Python REPL.
  2. Observe that both return True.

Environment:

Library version:
Python version: Python 3.10.14
OS: Windows

Fix? :

  • Add port range validation and treat bare hostnames with paths as invalid
  • unless explicitly allowed by the spec.

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne damit, beide Beispiele in einer Python-REPL nachzustellen und nachzuverfolgen, wie uri_reference(...).is_valid() Hostnamen und Ports validiert. Als abgeschlossen gilt, wenn einfache Hostnamen mit Pfaden und Ports über 65535 abgelehnt werden und Tests beide gemeldeten Eingaben abdecken.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
python
Bereich
networking
Issue-Typ
Bug
Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.