List of UDP ports included in UDP port scan

[duncangreene]

Describe the issue
Where is it defined (maybe in code and/or documentation) what ports are scanned in the UDP scan? I was surprised to see that UDP 47808 was not listed in the Services section of the report for a BACnet/IP module.

Note below showing BACnet service running.

To Reproduce
Steps to reproduce the behavior:

1. Run a Services check against a DUT running a UDP service, for example any BACnet/IP device.

Expected behavior
UDP 47808 should appear in the list of services, and the "UDP ports open" count should reflect this.

Environment (please provide the following information about your setup):

• Testrun 2.0

report (11).zip

[duncangreene]

I believe the below line should print which UDP ports are to be scanned...

...but this line does not appear in my Testrun logs despite log level = debug (logs available in OP)...

However the UDP scan does show complete 6 seconds later, which seems quite quick for a UDP scan. I've also seen one finish in 2 seconds below.

[jboddey]

It looks like the test container is not printing that line because the log level within the container is at INFO rather than DEBUG. We should probably change that line to INFO. In addition, BACnet will be added in the UDP scan. However, we can't add all ports as the scan would take too long.

[jboddey]

This has now been merged into dev and will be released in v2.1.

[duncangreene]

@jboddey Just confirming that in 2.1 I still do not see port 47808 included in the UDP scan as below.

The below device definitely has ports TCP 502 and UDP 47808 open at all times.

P.S. FWIW, I would advocate that UDP 3671 for KNXnet/IP is included too if there is a desire to restrict to 'common known ports'.
P.P.S. I don't believe it should make a difference, but the above screenshots were taken from a test with protocol.* tests disabled. Edit. I just reran with protocol.* tests enabled but no difference.

[duncangreene]

Just confirming this is still present on 2.1.1. UDP 47808 should be shown for the device below.

[MariusBaldovin]

@duncangreene Are you able to share the full zip?
Can you check if you have protocol.services.bacnet test included in the qualification.json and pilot.json files from resources/test_packs/