sindresorhus/conf

Support using a Conf store from multiple processes

Open

#146 aberto em 11 de mar. de 2021

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)TypeScript (1.340 stars) (138 forks)user submission
enhancementhelp wanted

Description

Currently if you have multiple processes using conf with the same storage location they can, and do, clobber each other's data.

e.g.

  1. p1 reads store
  2. p2 reads store
  3. p1 writes store
  4. p2 writes store (this clobbers p1's changes in step 3, even if the change was a totally different key)

Any suggestions on how to work around this?

I feel like this could warrant a big disclaimer in the readme.

Guia do colaborador

Pilha de tecnologia
typescriptnodejs
Domain
tooling
Tipo Issue
feature
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
4
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 days
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
mostly clear
Prerequisites
Node.jsasync programmingfile system operations
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40
Direção de pesquisa
The issue describes a race condition when multiple processes read/write the same config store. A solution likely requires file locking or atomic writes. Investigate Node.js `fs` module for file locking mechanisms (e.g., `fs.open` with exclusive flag) or use a library like `proper lockfile`. The existing code uses `fs.writeFileSync` and `fs.readFileSync`; consider switching to `writeFile` with atomic rename. Look at issue comments for any suggested approaches.