eclipse-theia/theia

URL path escapes '@' symbol, which breaks the url

Open

#4927 aperta il 17 apr 2019

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (2478 fork)batch import
help wanted

Metriche repository

Star
 (18.676 star)
Metriche merge PR
 (Merge medio 15g 4h) (75 PR mergiate in 30 g)

Descrizione

Description

I have theia running under JupterLab, the username in the url is an email address and contains an @, for example:

/users/rayh@example.com/theia

While I can load the page, the websocket url escapes the @ and therefore breaks the url and the websocket connection cannot be established.

In webpack:///./node_modules/vscode-uri/lib/esm/index.js - if I flip skipEncoding to true, the url now works, but i suspect this will not encode the request params. image

Here is the failing connection: image

Reproduction Steps

Simplest test would be to deploy theia under a url path that contains an '@'

OS and Theia version: Theia 0.5.0 running in Ubutnu 18.04.1 LTS

Guida contributor

Direzione di ricerca
Individua la logica di codifica URL nel codebase, specialmente in vscode uri o codice simile per la gestione degli URI. Indaga perché il carattere '@' viene codificato quando non dovrebbe esserlo nei segmenti di percorso.
Tech stack
typescriptnodejs
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65