Skip to content

Complete work to expose security protocols#229

Merged
abeltrano merged 8 commits intodevelopfrom
wpactrlpath
Mar 18, 2024
Merged

Complete work to expose security protocols#229
abeltrano merged 8 commits intodevelopfrom
wpactrlpath

Conversation

@abeltrano
Copy link
Contributor

Type

  • Bug fix
  • Feature addition
  • Feature update
  • Documentation
  • Build Infrastructure

Side Effects

  • Breaking change
  • Non-functional change

Goals

  • Ensure security protocols are provided in capability information.
  • Ensure security protocols are provided in cli output for wifi enumaps command.

Technical Details

  • Combine all security protocols to a single line for ToString() output.
  • Define constants for cipher suites and akms assoicated with each security protocol.
  • Move Ieee80211 <-> wiphy adapters to libnl-helpers library where they belong.
  • Add list of supported nl80211_wpa_versions
  • Update default wpa/hostapd control socket search directory.

Test Results

  • All unit tests pass.
  • wifi enumap command now shows security protocols:
image

Reviewer Focus

  • None

Future Work

  • Determine if WPA versions can be indirectly obtained without using the NL80211_ATTR_WPA_VERSIONS attribute selector.

Checklist

  • Build target all compiles cleanly.
  • clang-format and clang-tidy deltas produced no new output.
  • Newly added functions include doxygen-style comment block.

@abeltrano abeltrano requested a review from a team as a code owner March 18, 2024 20:58
@abeltrano abeltrano merged commit 6722897 into develop Mar 18, 2024
@abeltrano abeltrano deleted the wpactrlpath branch March 18, 2024 21:28
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