actualbudget / actualbudget/actual

[Bug]: Installing npm @actual-app/sync-server ends up with warnings/errors

Aberta
#8,584 3 comentários 1 reação 0 responsáveis Ver no GitHub
good first issue server
Linguagem predominante
TypeScript
Estrelas
28.7k
Forks
3k
Merge médio
2d 11h
PRs com merge (30d)
65

Descrição

### What happened?

I tried to install the server package by running `npm install -g @actual-app/sync-server` but I end up with the following warnings that seem to be critical in the end:

```
npm warn allow-scripts 2 packages have install scripts not yet covered by allowScripts:
npm warn allow-scripts bcrypt@6.0.0 (install: node-gyp-build)
npm warn allow-scripts better-sqlite3@12.11.1 (install: prebuild-install || node-gyp rebuild --release)
npm warn allow-scripts
npm warn allow-scripts Run `npm approve-scripts --allow-scripts-pending` to review, or `npm approve-scripts ` to allow.
```

I guess I can workaround using `--allow-scripts=bcrypt,better-sqlite3` but it sounds like `allowScripts` should be added to actual's package.json, right?

Edit: blocking install scripts is now default in NPM 12, that why this issue did not occur before.
Edit2: 26.8.0 adds `argon2` to the list

### How can we reproduce the issue?

1. run `npm install -g @actual-app/sync-server`

### Where are you hosting Actual?

Locally via Yarn

### What browsers are you seeing the problem on?

Other

### Operating System

Linux

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.