Commit bb2287d
fix(bindings): thread adminToken through PilotPolicySet (#14)
common's driver.PolicySet gained a third `adminToken string` parameter
in v0.4.4. libpilot's `bindings.go` was still calling the old 2-arg
form, so the whole CGO shim has been failing to build on main since
2026-06-06.
Adds a fourth C argument (`adminToken *C.char`) to PilotPolicySet,
mirroring the convention PilotBroadcast already follows. Empty
strings are valid — the daemon side treats an absent token the same
as "no admin auth attempted."
This is a C ABI break, but the library has been uncompilable for
three days; anyone embedding it can't be. Re-publishing as v0.x with
the new symbol shape is the right call.
go.mod tidy also bumps the sibling deps that drifted alongside:
common 0.4.3 → 0.4.8
runtime 0.1.0 → 0.3.1
handshake 0.1.0 → 0.2.1
policy 0.1.0 → 0.2.2
trustedagents 0.1.0 → 0.2.3
Build + unit tests pass.
Co-authored-by: Teodor Calin <teodor@vulturelabs.io>1 parent 2e2208e commit bb2287d
3 files changed
Lines changed: 10 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
718 | 718 | | |
719 | 719 | | |
720 | 720 | | |
721 | | - | |
| 721 | + | |
722 | 722 | | |
723 | 723 | | |
724 | 724 | | |
725 | 725 | | |
726 | | - | |
| 726 | + | |
727 | 727 | | |
728 | 728 | | |
729 | 729 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
| 7 | + | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | 13 | | |
14 | | - | |
15 | | - | |
16 | | - | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
17 | 18 | | |
18 | 19 | | |
19 | 20 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
| 5 | + | |
| 6 | + | |
0 commit comments