SVI/ProxmoxVE
1
0
Fork 0
mirror of https://github.com/community-scripts/ProxmoxVE.git synced 2026-07-24 14:53:58 -04:00
Code Issues 2.6k Packages Projects Releases 10 Wiki Activity

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

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

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_
saavagebueno added the bugin project pipelinerefactor labels 2025-11-20 04:52:50 -05:00
saavagebueno commented 2025-11-20 04:52:51 -05:00
Author
Owner
Copy Link

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

Same issue with homarr today

@AlkaizerLord commented on GitHub (Mar 16, 2025): Same issue with homarr today
saavagebueno commented 2025-11-20 04:52:51 -05:00
Author
Owner
Copy Link

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

same issue with zipline

@andyad8865 commented on GitHub (Mar 20, 2025): same issue with zipline
saavagebueno commented 2025-11-20 04:52:51 -05:00
Author
Owner
Copy Link

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

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

done

@MickLesk commented on GitHub (May 6, 2025): done
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 in project pipeline refactor
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#718
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?