akka/akka-http

empty Uri fragment ignored

Open

#3 859 ouverte le 29 juin 2021

Voir sur GitHub
 (7 commentaires) (0 réactions) (0 assignés)Scala (598 forks)batch import
1 - triagedhelp wantedt:coret:model

Métriques du dépôt

Stars
 (1 311 stars)
Métriques de merge PR
 (Merge moyen 1h 17m) (1 PR mergée en 30 j)

Description

The Uri.withFragment function does not allow one to set an empty fragment, which is allowed by the IETF spec I believe, and by the Uri constructor, as shown:

import akka.http.scaladsl.model.Uri
import Uri.*

scala> val good = Uri("https",Authority(Host("example.com")),Path./("doc"),None,Some(""))       
val good: akka.http.scaladsl.model.Uri = https://example.com/doc#

scala> val noFrag = Uri("https",Authority(Host("example.com")),Path./("doc"),None,None)
val noFrag: akka.http.scaladsl.model.Uri = https://example.com/doc

scala> noFrag.withFragment("")
val res3: akka.http.scaladsl.model.Uri = https://example.com/doc

Guide contributeur

Direction de recherche
Le problème se trouve dans la méthode `withFragment` de la classe `Uri` dans Akka HTTP. L'implémentation actuelle traite une chaîne vide comme aucun fragment, mais le constructeur URI autorise un fragment vide. Pour corriger, examinez `Uri.scala` autour de la méthode `withFragment` (probablement dans `akka http core/src/main/scala/akka/http/scaladsl/model/Uri.scala`). La correction doit garantir que définir une chaîne de fragment vide donne un URI avec un fragment vide, pas `None`. Vérifiez la logique du constructeur pour savoir comment le fragment est stocké et adaptez `withFragment` en conséquence. De plus, consultez la spécification IETF URI (RFC 3986) pour confirmer que le fragment vide est autorisé.
Stack technique
scala
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
80