Commit fb7759b
authored
* feat: Implement server CLI interface (bssh-server binary) (#131)
Implement the bssh-server binary with comprehensive CLI interface for
managing the SSH server.
Features:
- Main commands: run, gen-config, hash-password, check-config,
gen-host-key, version
- Global CLI arguments: -c/--config, -b/--bind-address, -p/--port,
-k/--host-key, -v/--verbose, -D/--foreground, --pid-file
- Configuration file loading with CLI overrides
- Signal handling for graceful shutdown (SIGTERM, SIGINT)
- Password hashing with bcrypt (cost factor 12)
- SSH host key generation (Ed25519/RSA)
- Configuration validation and checking
- Proper error handling and exit codes
Technical details:
- Uses clap for CLI parsing with derive API
- Integrates with existing ServerFileConfig from issue #130
- Supports both file-based and CLI-based configuration
- Added dependencies: bcrypt 0.16, rand 0.8, ssh-key 0.6
- All clippy checks pass with -D warnings
Resolves #131
* fix: Address critical and high security issues in server CLI
- Fix host key file race condition by using atomic file creation with mode 0o600
- Add exclusive PID file lock check to prevent multiple server instances
- Add password complexity warning for passwords shorter than 8 characters
- Set restrictive permissions (0600) on generated config files
* test: Add comprehensive tests for server CLI binary
- Add 19 new unit tests covering CLI parsing, subcommands, and options
- Test gen-config with file output and permissions validation
- Test gen-host-key for Ed25519/RSA with permission checks
- Test write_pid_file including stale PID handling
- Test all CLI parsing scenarios and global options
- Update ARCHITECTURE.md with Server CLI Binary documentation
- Update docs/architecture/README.md with server CLI references
- Update docs/architecture/server-configuration.md with CLI commands section
- Fix formatting issues in bssh_server.rs (cargo fmt)
1 parent 0a42a3e commit fb7759b
File tree
7 files changed
+1089
-4
lines changed- docs/architecture
- src/bin
7 files changed
+1089
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
189 | 189 | | |
190 | 190 | | |
191 | 191 | | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
192 | 238 | | |
193 | 239 | | |
194 | 240 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
54 | 54 | | |
55 | 55 | | |
56 | 56 | | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
57 | 60 | | |
58 | 61 | | |
59 | 62 | | |
| |||
72 | 75 | | |
73 | 76 | | |
74 | 77 | | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
33 | 33 | | |
34 | 34 | | |
35 | 35 | | |
| 36 | + | |
36 | 37 | | |
37 | 38 | | |
38 | 39 | | |
| |||
59 | 60 | | |
60 | 61 | | |
61 | 62 | | |
| 63 | + | |
62 | 64 | | |
63 | 65 | | |
64 | 66 | | |
| |||
70 | 72 | | |
71 | 73 | | |
72 | 74 | | |
| 75 | + | |
| 76 | + | |
73 | 77 | | |
74 | 78 | | |
75 | 79 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
335 | 335 | | |
336 | 336 | | |
337 | 337 | | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
| 346 | + | |
| 347 | + | |
| 348 | + | |
| 349 | + | |
| 350 | + | |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
| 361 | + | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
| 375 | + | |
| 376 | + | |
| 377 | + | |
| 378 | + | |
| 379 | + | |
| 380 | + | |
| 381 | + | |
| 382 | + | |
| 383 | + | |
| 384 | + | |
| 385 | + | |
| 386 | + | |
| 387 | + | |
| 388 | + | |
| 389 | + | |
| 390 | + | |
| 391 | + | |
| 392 | + | |
| 393 | + | |
| 394 | + | |
| 395 | + | |
| 396 | + | |
| 397 | + | |
338 | 398 | | |
339 | 399 | | |
340 | 400 | | |
| 401 | + | |
341 | 402 | | |
342 | 403 | | |
343 | 404 | | |
| |||
0 commit comments