sarman/tftsr-devops_investigation
1
1
Fork 0
Code Issues 4 Pull Requests 1 Actions 34 Packages Projects 3 Releases 6 Wiki Activity

fix: resolve Proxmox authentication response parsing error #126

Merged
sarman merged 4 commits from fix/proxmox-auth-decoding into beta 2026-06-21 03:13:38 +00:00
Conversation 0 Commits 4 Files Changed 10 +5015 -16186

4 Commits

Author SHA1 Message Date
Shaun Arman
bad3042d2d style: format proxmox client code with cargo fmt
Some checks failed
PR Review Automation / review (pull_request) Has been cancelled
Details
Test / frontend-tests (pull_request) Successful in 1m46s
Details
Test / frontend-typecheck (pull_request) Successful in 1m53s
Details
Test / rust-fmt-check (pull_request) Successful in 12m1s
Details
Test / rust-clippy (pull_request) Successful in 13m25s
Details
Test / rust-tests (pull_request) Successful in 15m37s
Details
2026-06-20 21:56:49 -05:00
Shaun Arman
cec962b349 docs: remove hardcoded 172.0.0.18 references
Some checks failed
PR Review Automation / review (pull_request) Has been cancelled
Details
Test / frontend-typecheck (pull_request) Has been cancelled
Details
Test / frontend-tests (pull_request) Has been cancelled
Details
Test / rust-tests (pull_request) Has been cancelled
Details
Test / rust-clippy (pull_request) Has been cancelled
Details
Test / rust-fmt-check (pull_request) Has been cancelled
Details 
Replace with generic proxmox-server hostname across all documentation
and code comments.
 2026-06-20 21:51:51 -05:00
Shaun Arman
d51dcc102f refactor(proxmox): extract test helper and use PROXMOX_HOST env var 
- Add get_test_client() helper to reduce test duplication
- Use PROXMOX_HOST environment variable for configurable server address
- Add PROXMOX_HOST to ~/.bashrc with default value
 2026-06-20 21:49:18 -05:00
Shaun Arman
1904f832c6 fix: resolve Proxmox authentication response parsing error
Some checks failed
Test / frontend-tests (pull_request) Successful in 1m41s
Details
Test / frontend-typecheck (pull_request) Successful in 1m51s
Details
PR Review Automation / review (pull_request) Successful in 4m24s
Details
Test / rust-fmt-check (pull_request) Failing after 12m43s
Details
Test / rust-clippy (pull_request) Successful in 13m51s
Details
Test / rust-tests (pull_request) Successful in 15m12s
Details 
- Removed incorrect #[serde(rename_all = "PascalCase")] attribute from AuthResponse struct
- Proxmox API returns lowercase fields (ticket, username, clustername) not PascalCase
- Added missing clustername field to AuthResponse struct
- Updated unit tests to match actual Proxmox API response format
- Added 4 integration tests for Proxmox API endpoints:
  * test_real_proxmox_auth - verifies authentication works
  * test_real_proxmox_cluster_resources - fetches cluster resources
  * test_real_proxmox_nodes - fetches node status
  * test_real_proxmox_vms - fetches VM list
- All 432 Rust tests passing
- All integration tests verified against https://172.0.0.18:8006
 2026-06-20 21:37:39 -05:00