eclipse-theia/theia

Workspace service does not handle updates on network drives

Open

#12.132 aperta il 1 feb 2023

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)TypeScript (2478 fork)batch import
help wantedworkspace

Metriche repository

Star
 (18.676 star)
Metriche merge PR
 (Merge medio 12g 10h) (73 PR mergiate in 30 g)

Descrizione

Bug Description:

If I create a multiroot workspace on a network drive then Workspace service couldn't handle workspace root updates. https://github.com/eclipse-theia/theia/blob/b28c8b9d543e7df6c5d2bc524d40c50414df7e4d/packages/workspace/src/browser/workspace-service.ts#L104 "event.contains(this._workspace.resource)" Condition is always false because this._workspace.resource == "/u:/my-workspace" and event.changes[0].resource == "/userbackup/admin/my-workspace/roots.json" But u:/ and /userbackup/admin are the same drives. Nevertheless, "event.contains" check returns false

Steps to Reproduce:

  1. Create multiroot workspace on a network drive where
  2. Open ide on this workspace
  3. Edit the file containing workspace roots
  4. Workspace service couldn't handle file update

Additional Information

  • Operating System: Windows 10

Guida contributor

Direzione di ricerca
Il problema si trova in workspace service.ts alla riga 104, dove event.contains verifica l'uguaglianza degli URI, ma i percorsi delle unità di rete differiscono (ad esempio, 'u:/' rispetto a '/userbackup/admin'). Indaga come vengono costruiti gli URI su Windows e considera l'uso di fs.realpathSync o di un'utilità di normalizzazione dei percorsi. Controlla le utility URI di Theia per la gestione dei percorsi cross platform.
Tech stack
typescriptnodejs
Dominio
backenddesktop
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
Node.jsTypeScriptFile system path handling on Windows
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40