Skip to content

Fix Proxmox discovery for sudo users - #1379

Open
ZacharyZcR wants to merge 1 commit into
Termix-SSH:dev-2.8.0from
ZacharyZcR:fix/support-1246-proxmox-sudo
Open

Fix Proxmox discovery for sudo users#1379
ZacharyZcR wants to merge 1 commit into
Termix-SSH:dev-2.8.0from
ZacharyZcR:fix/support-1246-proxmox-sudo

Conversation

@ZacharyZcR

Copy link
Copy Markdown
Member

Summary

  • run every Proxmox discovery pvesh read through the existing elevated-command path when the host has a stored sudo password
  • preserve direct execution for root hosts without sudo credentials
  • cover forced elevation and failed-command propagation with focused regression tests

Fixes Termix-SSH/Support#1246

Validation

  • npx vitest run --project backend src/backend/tests/database/routes/proxmox-sudo.test.ts src/backend/tests/hosts/metrics/managers/exec-elevated.test.ts
  • npx eslint src/backend/database/routes/proxmox.ts src/backend/tests/database/routes/proxmox-sudo.test.ts
  • npx prettier --check src/backend/database/routes/proxmox.ts src/backend/tests/database/routes/proxmox-sudo.test.ts
  • npx tsc --noEmit
  • git diff --check

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.

1 participant