-
Notifications
You must be signed in to change notification settings - Fork 39
dot1x: add macOS and Windows 802.1X supplicant state table #113
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
Open
robbiet480
wants to merge
37
commits into
macadmins:main
Choose a base branch
from
robbiet480:add-eapol-status
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
f4f3334
eapol_status: add macOS 802.1X/EAPOL supplicant state table
robbiet480 fbe964e
CI: gate darwin binary builds behind macOS host platform check
robbiet480 7281095
CI: gate mac binary cquery in bazel_to_builddir.sh behind macOS host …
robbiet480 e5898f1
CI: scope `make test` to only go_test targets, not all targets
robbiet480 d7bbdfe
CI: exclude eapolstatus from go-test-coverage on Linux
robbiet480 e2ca4d1
eapol_status: add Windows backend using wlanapi.dll
robbiet480 6628f2c
docs: update README eapol_status entry for Windows support
robbiet480 7230e49
eapol_status: add Windows unit tests and mock tests
robbiet480 463bf1c
Merge branch 'main' into add-eapol-status
robbiet480 b5e89b2
coverage: narrow eapolstatus exclusion to platform-specific files only
robbiet480 7fd7b48
CI: broaden make test query and harden bazel_to_builddir.sh
robbiet480 34aa32c
eapol_status: add macOS mock-backend tests
robbiet480 e0612d9
Rename eapol_status table to dot1x
robbiet480 c11ba8e
Address Copilot review: neutral backend-unavailable sentinel, robust …
robbiet480 cf97ff7
dot1x: strip all whitespace from TrustedRootCA hex, not just spaces
robbiet480 7189887
dot1x: reset SupplicantState for non-802.1X Wi-Fi networks
robbiet480 59f2378
dot1x: use syscall.Errno for Win32 errors, windows.UTF16PtrToString, …
robbiet480 1de369b
dot1x: pass opcodeValueType out-param to WlanQueryInterface, capture …
robbiet480 2971348
dot1x: skip live tests on ErrBackendUnavailable, clarify cgo comments
robbiet480 07ff06f
Address Copilot review: cover Linux backend, distinguish cquery failures
robbiet480 811d82c
dot1x(windows): reuse WLAN handle + parse profile XML via encoding/xml
robbiet480 207f740
dot1x(windows): skip on query failure, gate live tests, rebind loop var
robbiet480 ed45470
Address Copilot review: portable mktemp, dup interface keys, bounded …
robbiet480 5023bad
Address Copilot review: cp -- , validate TrustedRootCA hex, guard for…
robbiet480 c3afda4
Address Copilot review: portable cp option-guard, well-formed namespa…
robbiet480 68c6478
Address Copilot review: empty-vs-nil defaults, dlerror detail, robust…
robbiet480 fcb3142
dot1x(windows): harden syscall error paths; move WLAN XML parsing to …
robbiet480 cfe7a1d
dot1x: error on NULL status (darwin), wrap underlying enum error (win…
robbiet480 7e1e2a2
dot1x tests: use context.Background(); BUILD: tag mac cgo binaries ma…
robbiet480 7e63e2d
dot1x: accurate Dot1XBackend doc; de-dup constraintFor test helper
robbiet480 d70376d
dot1x(windows): system DLL load, separate trusted-root-CA column, sin…
robbiet480 6891d45
dot1x(windows): report specific init failure; Makefile: robust uname …
robbiet480 3dfd943
dot1x: make TestInterfacesToQuery deterministic via interfaceLister stub
robbiet480 da811c5
dot1x: fix double-enum in interfacesToQuery; parse WLAN profile in on…
robbiet480 e968c8e
dot1x/build: robust EapMethod scan, right-sized hex buf, trap temp-fi…
robbiet480 c47befc
bazel_to_builddir.sh: scope temp-file cleanup to a subshell EXIT trap
robbiet480 f9087b9
bazel_to_builddir.sh: tolerate missing uname; BUILD: fix stale gating…
robbiet480 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,55 @@ | ||
| load("@io_bazel_rules_go//go:def.bzl", "go_library", "go_test") | ||
|
|
||
| go_library( | ||
| name = "dot1x", | ||
| srcs = [ | ||
| "dot1x.go", | ||
| "dot1x_darwin.go", | ||
| "dot1x_other.go", | ||
| "dot1x_wlanprofile.go", | ||
| "dot1x_windows.go", | ||
| ], | ||
| # cgo is only needed on darwin: dot1x_darwin.go (//go:build darwin) calls | ||
| # EAP8021X.framework (via dlopen/dlsym) + CoreFoundation via cgo. | ||
| # The darwin build constraint does NOT include ios, so only darwin gets | ||
| # the cgo + framework links; linux/windows stay pure Go. | ||
| cgo = select({ | ||
| "@io_bazel_rules_go//go/platform:darwin": True, | ||
| "//conditions:default": False, | ||
| }), | ||
| clinkopts = select({ | ||
| "@io_bazel_rules_go//go/platform:darwin": [ | ||
| "-framework", | ||
| "CoreFoundation", | ||
| ], | ||
| "//conditions:default": [], | ||
| }), | ||
|
robbiet480 marked this conversation as resolved.
robbiet480 marked this conversation as resolved.
robbiet480 marked this conversation as resolved.
|
||
| importpath = "github.com/macadmins/osquery-extension/tables/dot1x", | ||
| visibility = ["//visibility:public"], | ||
| deps = [ | ||
| "@com_github_osquery_osquery_go//plugin/table", | ||
| ] + select({ | ||
| "@io_bazel_rules_go//go/platform:windows": [ | ||
| "@org_golang_x_sys//windows", | ||
| ], | ||
| "//conditions:default": [], | ||
| }), | ||
| ) | ||
|
|
||
| go_test( | ||
| name = "dot1x_test", | ||
| srcs = [ | ||
| "dot1x_test.go", | ||
| "dot1x_darwin_test.go", | ||
| "dot1x_helpers_test.go", | ||
| "dot1x_other_test.go", | ||
| "dot1x_wlanprofile_test.go", | ||
| "dot1x_windows_test.go", | ||
| ], | ||
| embed = [":dot1x"], | ||
| deps = [ | ||
| "@com_github_osquery_osquery_go//plugin/table", | ||
| "@com_github_stretchr_testify//assert", | ||
| "@com_github_stretchr_testify//require", | ||
| ], | ||
| ) | ||
|
robbiet480 marked this conversation as resolved.
|
||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.