eclipse-theia/theia

Workspace service does not handle updates on network drives

Open

#12.132 aberto em 1 de fev. de 2023

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)TypeScript (2.478 forks)batch import
help wantedworkspace

Métricas do repositório

Stars
 (18.676 stars)
Métricas de merge de PR
 (Mesclagem média 12d 10h) (73 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
O problema está em workspace service.ts na linha 104, onde event.contains verifica a igualdade de URI, mas os caminhos da unidade de rede diferem (por exemplo, 'u:/' vs '/userbackup/admin'). Investigue como os URIs são construídos no Windows e considere usar fs.realpathSync ou um utilitário de normalização de caminhos. Verifique os utilitários de URI do Theia para o tratamento de caminhos multiplataforma.
Pilha de tecnologia
typescriptnodejs
Domain
backenddesktop
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.
3
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.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Node.jsTypeScriptFile system path handling on Windows
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40