akka/akka-http

empty Uri fragment ignored

Offen

#3.859 geöffnet am 29.06.2021

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (598 Forks)batch import
1 - triagedhelp wantedt:coret:model

Repository-Metriken

Stars
 (1.311 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 1T 10h) (2 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Das Problem liegt in der `withFragment` Methode der `Uri` Klasse in Akka HTTP. Die aktuelle Implementierung behandelt einen leeren String als kein Fragment, aber der URI Konstruktor erlaubt ein leeres Fragment. Zur Behebung untersuchen Sie `Uri.scala` um die `withFragment` Methode herum (wahrscheinlich in `akka http core/src/main/scala/akka/http/scaladsl/model/Uri.scala`). Die Korrektur sollte sicherstellen, dass das Setzen eines leeren Fragment Strings zu einem URI mit einem leeren Fragment führt, nicht zu `None`. Überprüfen Sie die Konstruktorlogik, wie das Fragment gespeichert wird, und passen Sie `withFragment` entsprechend an. Überprüfen Sie außerdem die IETF URI Spezifikation (RFC 3986), um zu bestätigen, dass ein leeres Fragment erlaubt ist.
Tech Stack
scala
Domain
backend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80