akka/akka-http

empty Uri fragment ignored

Open

#3.859 aberto em 29 de jun. de 2021

Ver no GitHub
 (7 comments) (0 reactions) (0 assignees)Scala (598 forks)batch import
1 - triagedhelp wantedt:coret:model

Métricas do repositório

Stars
 (1.311 stars)
Métricas de merge de PR
 (Mesclagem média 1h 17m) (1 fundiu PR em 30d)

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

Guia do colaborador

Direção de pesquisa
O problema está no método `withFragment` da classe `Uri` no Akka HTTP. A implementação atual trata uma string vazia como nenhum fragmento, mas o construtor URI permite fragmento vazio. Para corrigir, examine `Uri.scala` em torno do método `withFragment` (provavelmente em `akka http core/src/main/scala/akka/http/scaladsl/model/Uri.scala`). A correção deve garantir que definir uma string de fragmento vazia resulte em um URI com um fragmento vazio, não `None`. Verifique a lógica do construtor para como o fragmento é armazenado e adapte `withFragment` de acordo. Além disso, revise a especificação IETF URI (RFC 3986) para confirmar que fragmento vazio é permitido.
Pilha de tecnologia
scala
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80