SVI/ProxmoxVE
1
0
Fork 0
mirror of https://github.com/community-scripts/ProxmoxVE.git synced 2026-03-31 06:34:17 -04:00
Code Issues 2.6k Packages Projects Releases 10 Wiki Activity

Nginx Proxy Manager ancient node & pnpm #1611

New Issue
Closed
opened 2025-11-20 05:13:19 -05:00 by saavagebueno · 4 comments
saavagebueno commented 2025-11-20 05:13:19 -05:00
Owner
Copy Link

Originally created by @burgerga on GitHub (Aug 1, 2025).

✅ 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?

  • 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 (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).

NA

🖼️ Additional context (optional).

No response

Originally created by @burgerga on GitHub (Aug 1, 2025). ### ✅ 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_
saavagebueno added the bug label 2025-11-20 05:13:19 -05:00
saavagebueno commented 2025-11-20 05:13:20 -05:00
Author
Owner
Copy Link

@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
saavagebueno commented 2025-11-20 05:13:20 -05:00
Author
Owner
Copy Link

@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...
saavagebueno commented 2025-11-20 05:13:20 -05:00
Author
Owner
Copy Link

@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.
saavagebueno commented 2025-11-20 05:13:20 -05:00
Author
Owner
Copy Link

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

Agreed! :D

@burgerga commented on GitHub (Aug 1, 2025): Agreed! :D
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
🛑 Failure to comply with the guidelines
breaking change
bug
bug
bugfix
deferred
delete script
dependencies
enhancement
external
feature
github
help wanted
Implemented in VED waiting push to Main
in project pipeline
invalid
investigation
json
maintenance
new script
new script
nice to have
not a script issue
not planned
organization
pull-request
Mirrored from GitHub Pull Request
 question
refactor
rename script
security
update script
website
wontdo
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
saavagebueno
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#1611
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?