akka/akka-http

Problem with query parameter separator

Open

#2.854 geöffnet am 9. Dez. 2019

Auf GitHub ansehen
 (8 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Scala (598 Forks)batch import
1 - triageddiscusshelp wantedt:coret:model

Repository-Metriken

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

Beschreibung

After moving Play Framework from version 2.4 to 2.7 and starting to use akka-http as the server we encountered one problem with some weird URLs that we were mistakenly generating.

This url http://localhost:9000/issue/1300068591272/_lines?projectId=22877&commitId=15565520 reached the server with query parameter called projectId and commitId while this url http://localhost:9000/issue/1300068591272/_lines?projectId=22877&commitId=15565520 does not get the second.

Seems like this might be related to the way akka-http parses URLs. Apparently the second URL is also valid because at least ; is a supported separator.

I would gladly fix this if you point me in the right direction.

References:

Contributor Guide

Research-Richtung
Untersuchen Sie, wie akka http Abfrageparameter parst, insbesondere die Behandlung von Semikolons als Trennzeichen gemäß RFC 3986 und HTML Spezifikation. Überprüfen Sie den Parsing Code für Abfragezeichenfolgen und stellen Sie sicher, dass sowohl '&' als auch ';' als Trennzeichen behandelt werden.
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.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
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.
Meist klar
Voraussetzungen
Scala
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60