Skip to content

Use getpass for secure password input in setup.py#9

Merged
jacobseunglee merged 2 commits into
refactor/claude-copilotfrom
copilot/sub-pr-6-another-one
Oct 29, 2025
Merged

Use getpass for secure password input in setup.py#9
jacobseunglee merged 2 commits into
refactor/claude-copilotfrom
copilot/sub-pr-6-another-one

Conversation

Copilot AI commented Oct 29, 2025

Copy link
Copy Markdown

Addresses security feedback from PR #6: password input was displayed in plain text during interactive setup.

Changes:

  • Import getpass module
  • Replace input() with getpass.getpass() for PROXMOX_PASSWORD field

Before:

env_vars['PROXMOX_PASSWORD'] = input("Proxmox password: ").strip()

After:

env_vars['PROXMOX_PASSWORD'] = getpass.getpass("Proxmox password: ").strip()

Password input is now hidden during entry, preventing shoulder-surfing and terminal history exposure.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: jacobseunglee <66867022+jacobseunglee@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on code refactor from PR #6 review comments Use getpass for secure password input in setup.py Oct 29, 2025
Copilot AI requested a review from jacobseunglee October 29, 2025 01:58
@jacobseunglee jacobseunglee marked this pull request as ready for review October 29, 2025 02:04
@jacobseunglee jacobseunglee merged commit 8d8c98c into refactor/claude-copilot Oct 29, 2025
@jacobseunglee jacobseunglee deleted the copilot/sub-pr-6-another-one branch October 29, 2025 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants