push.rocks/smartradius
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(imports): use node: built-in imports and remove dead smartpromise/smartdelay exports

Browse Source
This commit is contained in:
Jürgen Kunz
2026-02-11 15:57:37 +00:00
parent ec3440406f
commit 9c3f7acae8
8 changed files with 23 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
readme.hints.md
View File

@@ -57,7 +57,7 @@ ts/ (order: 3) - Main exports (re-exports server + client)
}
```
Node.js built-ins: `dgram` (UDP), `crypto` (MD5/HMAC)
Node.js built-ins: `node:dgram` (UDP), `node:crypto` (MD5/HMAC)
## Build System
- Uses `@git.zone/tsbuild` v4.x with tsfolders mode
@@ -107,5 +107,10 @@ Downloaded to `./spec/`:
- `rfc2865.txt` - RADIUS Authentication
- `rfc2866.txt` - RADIUS Accounting
## Code Quality Notes
- All Node.js built-in imports use `node:` prefix (ESM/Deno/Bun compatible)
- Dead `smartpromise`/`smartdelay` imports removed from `ts_client/plugins.ts` (packages kept in package.json)
- Rust migration assessed as not cost-effective: crypto ops already delegate to OpenSSL C, RADIUS packets are small (max 4096 bytes), IPC overhead would negate any gains
## Last Updated
2026-02-01 - Full implementation complete with RFC 2865/2866 compliance
2026-02-11 - Fixed bare node: imports, removed dead imports, assessed Rust migration