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

Support fast roaming protocol (802.11 k/v/r) #459

Merged

Conversation

LeadroyaL
Copy link
Contributor

@LeadroyaL LeadroyaL commented Jan 20, 2024

What does this implement/fix? Please describe.

Support fast roaming protocol (802.11 k/v/r).

Screenshot_2024-01-20-10-09-03-346_com vrem wifianalyzer BETA

222

Additional context

feature Element check bit
802.11k RM Enabled Capabilities Neighbor Report
802.11v Extended Capabilities BSS Transition
802.11r Mobile Domain any

Reference: 802.11-2012 8.4.2 Information elements

Where has this been tested?

Yes, with unittest and on-board test

  • Operating System: Android 9/10/12
  • Platform: Redmi 8A, Samsung A60, Redmi 12 Turbo
  • Target Platform: Android
  • Toolchain Version: Any
  • SDK Version: 33

@LeadroyaL LeadroyaL marked this pull request as ready for review January 20, 2024 05:13
Copy link

codecov bot commented Jan 20, 2024

Codecov Report

Merging #459 (1caca19) into feature (f42bf38) will increase coverage by 0.06%.
Report is 4 commits behind head on feature.
The diff coverage is 97.72%.

Additional details and impacted files
@@              Coverage Diff              @@
##             feature     #459      +/-   ##
=============================================
+ Coverage      92.92%   92.98%   +0.06%     
- Complexity       922      931       +9     
=============================================
  Files            131      132       +1     
  Lines           2726     2767      +41     
  Branches         221      230       +9     
=============================================
+ Hits            2533     2573      +40     
  Misses            58       58              
- Partials         135      136       +1     

@VREMSoftwareDevelopment
Copy link
Owner

@LeadroyaL
Can you please fix the issues reported by Codecov?
#459 (comment)

@LeadroyaL
Copy link
Contributor Author

@VREMSoftwareDevelopment done

@VREMSoftwareDevelopment
Copy link
Owner

@LeadroyaL

  • Please resolve conflicts in PR.
  • Please merge the latest feature branch into your working branch.

Thanks

@LeadroyaL
Copy link
Contributor Author

@VREMSoftwareDevelopment done

@VREMSoftwareDevelopment VREMSoftwareDevelopment changed the base branch from feature to fast-roaming February 10, 2024 16:17
@VREMSoftwareDevelopment VREMSoftwareDevelopment merged commit eb4db49 into VREMSoftwareDevelopment:fast-roaming Feb 10, 2024
5 checks passed
@VREMSoftwareDevelopment
Copy link
Owner

Thank you for your contribution

VREMSoftwareDevelopment added a commit that referenced this pull request Feb 14, 2024
* add fast roaming information

* add test for fast roaming information

* add information for earlier android version

* improve code coverage

* Add boundary check, avoid package from evil access point.

* Move `FastRoaming` to separate file.

* Avoid null in kotlin.

Co-authored-by: LeadroyaL <LeadroyaL@users.noreply.github.com>
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.

2 participants