akka/akka-http

empty Uri fragment ignored

Aperta

#3859 aperta il 29 giu 2021

Vedi su GitHubGuida contributor
 (7 commenti) (0 reazioni) (0 assegnatari)Scala (598 fork)batch import
1 - triagedhelp wantedt:coret:model

Metriche repository

Star
 (1311 stelle)
Metriche merge PR
 (Merge medio 1g 10h) (2 PR mergiate in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Il problema si trova nel metodo `withFragment` della classe `Uri` in Akka HTTP. L'implementazione corrente tratta una stringa vuota come nessun frammento, ma il costruttore URI consente un frammento vuoto. Per risolvere, esamina `Uri.scala` intorno al metodo `withFragment` (probabilmente in `akka http core/src/main/scala/akka/http/scaladsl/model/Uri.scala`). La correzione dovrebbe garantire che l'impostazione di una stringa di frammento vuota risulti in un URI con un frammento vuoto, non `None`. Controlla la logica del costruttore per come viene memorizzato il frammento e adatta `withFragment` di conseguenza. Inoltre, rivedi la specifica IETF URI (RFC 3986) per confermare che il frammento vuoto sia consentito.
Tech stack
scala
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80