starred/ProxmoxVE

Fork 0

mirror of https://github.com/community-scripts/ProxmoxVE.git synced 2026-04-25 09:15:59 +03:00

[GH-ISSUE #6465] Nginx Proxy Manager ancient node & pnpm #1376

New issue

Closed

opened 2026-02-26 12:48:41 +03:00 by kerem · 4 comments

kerem commented

2026-02-26 12:48:41 +03:00

Owner

Originally created by @burgerga on GitHub (Aug 1, 2025).
Original GitHub issue: https://github.com/community-scripts/ProxmoxVE/issues/6465

✅ Have you read and understood the above guidelines?

yes

📜 What is the name of the script you are using?

nginxproxymanager

📂 What was the exact command used to execute the script?

⚙️ What settings are you using?

Default Settings
Advanced Settings

🖥️ Which Linux distribution are you using?

Debian 12

📝 Provide a clear and concise description of the issue.

Currently

https://github.com/community-scripts/ProxmoxVE/blob/main/install/nginxproxymanager-install.sh#L48-L58

forces an ancient version of node & pnpm

The official images (a bit of a search) seem to use Node 20 (github.com/NginxProxyManager/docker-nginx-full@d055da8a90/docker/Dockerfile.certbot-node (L12)).

Propose to use

NODE_VERSION="20" NODE_MODULE="pnpm@latest" setup_nodejs

for Nginx Proxy Manager?

(and confusingly nginx proxy manager is abbreviated as npm, which makes searching for which npm is installed fun)

🔄 Steps to reproduce the issue.

type update in LXC, see ancient version of NPM being used

❌ Paste the full error output (if available).

🖼️ Additional context (optional).

No response

Originally created by @burgerga on GitHub (Aug 1, 2025). Original GitHub issue: https://github.com/community-scripts/ProxmoxVE/issues/6465 ### ✅ Have you read and understood the above guidelines? yes ### 📜 What is the name of the script you are using? nginxproxymanager ### 📂 What was the exact command used to execute the script? NA ### ⚙️ What settings are you using? - [x] Default Settings - [ ] Advanced Settings ### 🖥️ Which Linux distribution are you using? Debian 12 ### 📝 Provide a clear and concise description of the issue. Currently https://github.com/community-scripts/ProxmoxVE/blob/main/install/nginxproxymanager-install.sh#L48-L58 forces an ancient version of node & pnpm The official images (a bit of a search) seem to use Node 20 (https://github.com/NginxProxyManager/docker-nginx-full/blob/d055da8a901c6165c0213b0f25b6aa9f5bf1ab49/docker/Dockerfile.certbot-node#L12). Propose to use ``` NODE_VERSION="20" NODE_MODULE="pnpm@latest" setup_nodejs ``` for Nginx Proxy Manager? (and confusingly nginx proxy manager is abbreviated as npm, which makes searching for which npm is installed fun) ### 🔄 Steps to reproduce the issue. type `update` in LXC, see ancient version of NPM being used ### ❌ Paste the full error output (if available). NA ### 🖼️ Additional context (optional). _No response_

kerem

2026-02-26 12:48:41 +03:00

closed this issue
added the
bug
label

kerem commented

2026-02-26 12:48:42 +03:00

Author

Owner

@MickLesk commented on GitHub (Aug 1, 2025):

Doesnt work with node20, try this the last 6 months. Only work with 16 because there old deps
=> deprecated APIs (Buffer, crypto, etc.),
=> Build-Tool node-gyp failed

@MickLesk commented on GitHub (Aug 1, 2025): Doesnt work with node20, try this the last 6 months. Only work with 16 because there old deps => deprecated APIs (Buffer, crypto, etc.), => Build-Tool node-gyp failed

kerem commented

2026-02-26 12:48:42 +03:00

Author

Owner

@burgerga commented on GitHub (Aug 1, 2025):

Ah, that's a bummer, thanks for your fast reply! Weird that the official docker image uses node 20...

@burgerga commented on GitHub (Aug 1, 2025): Ah, that's a bummer, thanks for your fast reply! Weird that the official docker image uses node 20...

kerem commented

2026-02-26 12:48:42 +03:00

Author

Owner

@MickLesk commented on GitHub (Aug 1, 2025):

Only the certbot instance (extra instance)... so npm himself use 16. The repo mixes a lot of deps and versions, it's a mess.

@MickLesk commented on GitHub (Aug 1, 2025): Only the certbot instance (extra instance)... so npm himself use 16. The repo mixes a lot of deps and versions, it's a mess.

kerem commented

2026-02-26 12:48:42 +03:00

Author

Owner

@burgerga commented on GitHub (Aug 1, 2025):

Agreed! :D

@burgerga commented on GitHub (Aug 1, 2025): Agreed! :D

kerem referenced this issue

2026-02-26 13:32:33 +03:00

[PR #1376] [MERGED] Fix: ActualBudget Update-Function #3139

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/ProxmoxVE#1376

No description provided.

Rows
Columns