Fix SIP URI support for `urllib.parse.url{parse,split}` (RFC 3261)
Ouverte
Personne n'a encore pris cette issue.
stdlib
type-bug
- Langage dominant
- Python
- Étoiles
- 77.2k
- Forks
- 35.9k
- Métriques de merge des PR
- Métriques de PR en attente
Description
Bug report
Bug description:
urllib.parse currently supports the SIP URI scheme. However, it mislabels the netloc as the path.
>>> import urllib.parse
>>> urllib.parse.urlparse('sip:user:password@host:port;uri-parameters?headers')
ParseResult(scheme='sip', netloc='', path='user:password@host:port', params='uri-parameters', query='headers', fragment='')
RFC 3261 defines the sip URI without slashes, like so:
sip:user:password@host:port;uri-parameters?headers
The correct ParseResult would be:
>>> urllib.parse.urlparse('sip:user:password@host:port;uri-parameters?headers')
ParseResult(scheme='sip', netloc='user:password@host:port', path='', params='uri-parameters', query='headers', fragment='')
CPython versions tested on:
3.14
Operating systems tested on:
macOS
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser les points d’entrée urllib.parse.urlparse et urlsplit et reproduisez l’exemple de SIP URI de l’issue. Examinez le comportement de parsing existant et les tests pour les URLs spécifiques à un schéma, puis assurez-vous que les deux fonctions traitent l’authority SIP comme netloc et préservent les parameters et la query indiqués dans le résultat attendu.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- python
- Domaine
- networking
- Type d'issue
- Bug
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Activité
- À l'abandon
- Clarté
- Clairement spécifiée
- Accessibilité débutants
- 45/100