Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logSystemInfo parameter and USB VID/PID lookup #331

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bboudaoud-nv
Copy link
Collaborator

This branch adds support for logging SystemInfo to the output database as a new System_Info table. It also adds support for getting the VID/PID of all USB devices attached to the system and logging this as part of SystemInfo (which is also written to log.txt as it has been historically).

In order to get VID/PID this branch adds a dependency on setupapi.lib which will now need to be packaged with the project. This is currently a windows-only feature.

There is no support built-in for identifying mice/keyboards from other USB devices, but the VID/PID of every connected device is reported. An optional vid_pid.json file has been added to the data-files directory to provide a lookup of vendor/product names from the VID/PID pair. This file is produced using a Python web-scraping tool.

Merging this PR closes #288.

@bboudaoud-nv bboudaoud-nv added the enhancement New feature or request label Oct 4, 2021
@bboudaoud-nv bboudaoud-nv self-assigned this Oct 4, 2021
@jspjutNV jspjutNV added this to To Do in Issue Tracking via automation Oct 5, 2021
@jspjutNV jspjutNV modified the milestones: v21.11.01, v22.01.01 Oct 5, 2021
@jspjutNV
Copy link
Contributor

Is setupapi.lib a G3D project? This is a great idea, but we need to make sure we can package the library before we can merge this one.

@bboudaoud-nv bboudaoud-nv moved this from To Do to In progress in Issue Tracking May 6, 2022
@bboudaoud-nv bboudaoud-nv marked this pull request as ready for review May 25, 2022 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Issue Tracking
In progress
Development

Successfully merging this pull request may close these issues.

Collect information about current mouse
2 participants