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 and implement IHostapd::[Get/Set]Property() commands #32

Merged
merged 8 commits into from
Nov 23, 2023

Conversation

abeltrano
Copy link
Contributor

This pull request includes a variety of changes to the linux/wpa-controller codebase. The most important changes include adding new methods to the IHostapd interface, adding new test cases for the GetProperty() method of the Hostapd class, and updating the WpaCommand struct and related classes.

Main interface changes:

  • linux/wpa-controller/include/Wpa/IHostapd.hxx: Updated the IHostapd interface with new methods and changed the return type of the GetStatus() method.
  • linux/wpa-controller/include/Wpa/Hostapd.hxx: Added new member functions to the Hostapd class for controlling the hostapd daemon process.

Testing improvements:

  • tests/unit/linux/wpa-controller/TestHostapd.cxx: Added new test cases for the GetProperty() method of the Hostapd class.

Struct and class updates:

  • linux/wpa-controller/include/Wpa/WpaCommandSet.hxx: Added a new header file defining the WpaCommandSet struct.
  • linux/wpa-controller/include/Wpa/WpaCommand.hxx: Updated the WpaCommand struct and its constructor, and added a SetPayload() method. [1] [2] [3]
  • linux/wpa-controller/WpaCommandSet.cxx: Added a new source file implementing the constructor of the WpaCommandSet struct.

Other important changes:

  • linux/wpa-controller/CMakeLists.txt: Added new source and header files to the list of public headers and sources. [1] [2] [3]
  • linux/wpa-controller/include/Wpa/WpaCore.hxx: Added a new enum class representing the type of WPA daemon/service.
  • linux/wpa-controller/WpaController.cxx: Updated the SendCommand function to use the Payload member variable.

Note: Due to the length and complexity of the changes, only the top 7 most important changes are included in the list.

@abeltrano abeltrano requested a review from a team as a code owner November 23, 2023 04:05
@abeltrano abeltrano merged commit 389b2ed into develop Nov 23, 2023
2 checks passed
@abeltrano abeltrano deleted the wpagetsetcmds branch November 23, 2023 04:06
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