Skip to content

feat(common): define WLAN_REASON_IEEE_802_1X_AUTH_FAILED - #471

Merged
aloisklink merged 2 commits into
mainfrom
feat/define-WLAN_REASON_IEEE_802_1X_AUTH_FAILED
Mar 3, 2023
Merged

feat(common): define WLAN_REASON_IEEE_802_1X_AUTH_FAILED#471
aloisklink merged 2 commits into
mainfrom
feat/define-WLAN_REASON_IEEE_802_1X_AUTH_FAILED

Conversation

@aloisklink

Copy link
Copy Markdown
Contributor

Define WLAN_REASON_IEEE_802_1X_AUTH_FAILED, used by hostapd source-code, see https://w1.fi/cgit/hostap/tree/src/common/ieee802_11_defs.h?h=hostap_2_10#n213


Adapted from commit dff384c, except I've changed it to an enum instead of a #define since it's a bit cleaner that way (and I've added documentation).

Define `WLAN_REASON_IEEE_802_1X_AUTH_FAILED`,
used by hostapd source-code, see
https://w1.fi/cgit/hostap/tree/src/common/ieee802_11_defs.h?h=hostap_2_10#n213

I've used an enum instead of a `#define` since it's a bit cleaner.

Co-authored-by: Alexandru Mereacre <mereacre@gmail.com>
@aloisklink aloisklink added the enhancement New feature or request label Mar 3, 2023
@aloisklink aloisklink added this to the RADIUS ERP Server milestone Mar 3, 2023
@aloisklink aloisklink mentioned this pull request Mar 3, 2023
@codecov

codecov Bot commented Mar 3, 2023

Copy link
Copy Markdown

Codecov Report

Merging #471 (2d05b6e) into main (03bfc22) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #471   +/-   ##
=======================================
  Coverage   53.48%   53.48%           
=======================================
  Files         144      144           
  Lines       19963    19963           
=======================================
  Hits        10677    10677           
  Misses       9286     9286           
Impacted Files Coverage Δ
src/radius/common.h 42.59% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@aloisklink
aloisklink enabled auto-merge March 3, 2023 09:48
@aloisklink
aloisklink added this pull request to the merge queue Mar 3, 2023
Merged via the queue into main with commit 39baecc Mar 3, 2023
@aloisklink
aloisklink deleted the feat/define-WLAN_REASON_IEEE_802_1X_AUTH_FAILED branch March 3, 2023 10:04
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

None yet

Development

Successfully merging this pull request may close these issues.

2 participants