eclipse-theia/theia

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

Open

#4.927 geöffnet am 17. Apr. 2019

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (18.676 Stars) (2.478 Forks)batch import
help wanted

Beschreibung

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

Contributor Guide

Tech Stack
typescriptnodejs
Domain
full stack
Issue Type
bug
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.
under 1 hour
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic TypeScriptunderstanding of URL encoding
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
80
Research-Richtung
Examine the vscode uri library, specifically the file `lib/esm/index.js` where the `skipEncoding` flag is used. The issue suggests flipping it to `true` fixes the websocket URL, but may affect encoding of other request params. Investigate whether a localized fix can be applied to avoid global changes. Also look at Theia's websocket connection code to see how the URL is constructed and where the encoding occurs. Consider adding a test for URLs containing special characters like '@'.