Installation 'Cancel' button does not exit the script (in certain circumstances) #648

Closed
opened 2026-02-04 20:33:49 +03:00 by OVERLORD · 4 comments
Owner

Originally created by @marcusdoc on GitHub (Mar 16, 2025).

Originally assigned to: @MickLesk on GitHub.

Have you read and understood the above guidelines?

YES

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

NPMplus LXC

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

bash -c "$(wget -qLO - https://github.com/community-scripts/ProxmoxVE/raw/main/ct/npmplus.sh)"

📝 Provide a clear and concise description of the issue.

Using 'Advanced Settings'; when presented the option for 'OK' or 'Cancel'; the 'Cancel' button sometimes does not exit the script.
In the above referenced installation script (NOMplus LXC), the 'Cancel' button will do nothing in 'Container type' and 'root password', yet in 'Container ID' it will cancel the script (exit gracefully).

I also quickly tested the 'Nginx Proxy Manager LXC' installation script and replicated the same problem.

⚙️ What settings are you using?

  • Default Settings
  • Advanced Settings

🖥️ Which Linux distribution are you using?

Debian 12

🔄 Steps to reproduce the issue.

Step 1: In the PVE shell, enter the following command
bash -c "$(wget -qLO - https://github.com/community-scripts/ProxmoxVE/raw/main/ct/npmplus.sh)"

Step 2: Proceed with installation - 'YES'

Step 3: Settings - 'Advanced settings'

Step 4: 'OK' twice to proceed to next windows (bypass instructional information)

Step 5: Container Type - tab to 'Cancel' and press the enter key to confirm selection

Step 6: Script eventually loops user to same screen to select Container Type (does not gracefully exit the script)

Paste the full error output (if available).

(looping error)

🖼️ Additional context (optional).

No response

Originally created by @marcusdoc on GitHub (Mar 16, 2025). Originally assigned to: @MickLesk on GitHub. ### ✅ Have you read and understood the above guidelines? YES ### 📜 What is the name of the script you are using? NPMplus LXC ### 📂 What was the exact command used to execute the script? bash -c "$(wget -qLO - https://github.com/community-scripts/ProxmoxVE/raw/main/ct/npmplus.sh)" ### 📝 Provide a clear and concise description of the issue. Using 'Advanced Settings'; when presented the option for 'OK' or 'Cancel'; the 'Cancel' button sometimes does not exit the script. In the above referenced installation script (NOMplus LXC), the 'Cancel' button will do nothing in 'Container type' and 'root password', yet in 'Container ID' it will cancel the script (exit gracefully). I also quickly tested the 'Nginx Proxy Manager LXC' installation script and replicated the same problem. ### ⚙️ What settings are you using? - [ ] Default Settings - [x] Advanced Settings ### 🖥️ Which Linux distribution are you using? Debian 12 ### 🔄 Steps to reproduce the issue. Step 1: In the PVE shell, enter the following command bash -c "$(wget -qLO - https://github.com/community-scripts/ProxmoxVE/raw/main/ct/npmplus.sh)" Step 2: Proceed with installation - 'YES' Step 3: Settings - 'Advanced settings' Step 4: 'OK' twice to proceed to next windows (bypass instructional information) Step 5: Container Type - tab to 'Cancel' and press the enter key to confirm selection Step 6: Script eventually loops user to same screen to select Container Type (does not gracefully exit the script) ### ❌ Paste the full error output (if available). (looping error) ### 🖼️ Additional context (optional). _No response_
OVERLORD added the bugin project pipelinerefactor labels 2026-02-04 20:33:49 +03:00
Author
Owner

@AlkaizerLord commented on GitHub (Mar 17, 2025):

Same issue with homarr today

@AlkaizerLord commented on GitHub (Mar 17, 2025): Same issue with homarr today
Author
Owner

@andyad8865 commented on GitHub (Mar 20, 2025):

same issue with zipline

@andyad8865 commented on GitHub (Mar 20, 2025): same issue with zipline
Author
Owner

@michelroegl-brunner commented on GitHub (Mar 20, 2025):

Yeah its a known issue. We have a complete rebuild of the advanced settings in our roadmap.

@michelroegl-brunner commented on GitHub (Mar 20, 2025): Yeah its a known issue. We have a complete rebuild of the advanced settings in our roadmap.
Author
Owner

@MickLesk commented on GitHub (May 6, 2025):

done

@MickLesk commented on GitHub (May 6, 2025): done
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ProxmoxVE#648