[GH-ISSUE #26] Port check #19

Closed
opened 2026-02-26 17:32:09 +03:00 by kerem · 1 comment
Owner

Originally created by @patrickbuec on GitHub (Dec 5, 2024).
Original GitHub issue: https://github.com/Corsinvest/cv4pve-api-powershell/issues/26

What happened?

The script takes the target address and the port as input and checks the connection using a ping. However, since ping is not allowed in our case, but the port is open.

Expected behavior

Please check the port in line 105. Suggest replacing

if (Test-Connection -Ping $hostTmp -BufferSize 16 -Count 1 -ea 0 -quiet) {

with

if (Test-NetConnection -ComputerName $hostTmp -Port $portTmp) {

Relevant log output

No response

Proxmox VE Version

8.2.7

Version (bug)

8.2.3

Version (working)

No response

On what operating system are you experiencing the issue?

Windows

Pull Request

  • I would like to do a Pull Request
Originally created by @patrickbuec on GitHub (Dec 5, 2024). Original GitHub issue: https://github.com/Corsinvest/cv4pve-api-powershell/issues/26 ### What happened? The script takes the target address and the port as input and checks the connection using a ping. However, since ping is not allowed in our case, but the port is open. ### Expected behavior Please check the port in line 105. Suggest replacing ` if (Test-Connection -Ping $hostTmp -BufferSize 16 -Count 1 -ea 0 -quiet) {` with ` if (Test-NetConnection -ComputerName $hostTmp -Port $portTmp) {` ### Relevant log output _No response_ ### Proxmox VE Version 8.2.7 ### Version (bug) 8.2.3 ### Version (working) _No response_ ### On what operating system are you experiencing the issue? Windows ### Pull Request - [ ] I would like to do a Pull Request
kerem closed this issue 2026-02-26 17:32:10 +03:00
Author
Owner

@franklupo commented on GitHub (Dec 9, 2024):

Good idea. I'll check it out.

Best regards

<!-- gh-comment-id:2528027139 --> @franklupo commented on GitHub (Dec 9, 2024): Good idea. I'll check it out. Best regards
Sign in to join this conversation.
No labels
bug
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cv4pve-api-powershell#19
No description provided.