Skip to content

Conversation

@sylvain-pierrot
Copy link
Collaborator

@sylvain-pierrot sylvain-pierrot commented Aug 29, 2025

Summary

This PR implements a complete ION (Interplanetary Overlay Network) integration in the bp-socket daemon, with major architectural refactoring for better robustness and maintainability.

Key Changes

  • ION Integration : BpSAP management with bp_open/bp_close, dedicated threads for send/receive
  • Registry Architecture : SAP and ADU registries with linked lists instead of fixed arrays
  • Netlink Confirmation : Kernel waits for daemon confirmation before proceeding
  • Thread Management : Conditional bp_interrupt and clean thread lifecycle
  • Code Quality : Error handling with goto, improved logging, better function names

… interfaces (no behavior change)

Signed-off-by: Sylvain Pierrot <[email protected]>
Signed-off-by: Sylvain Pierrot <[email protected]>
- Implement BpSAP registry with linked list management
- Add conditional bp_interrupt for endpoint closure
- Refactor ADU management into registry pattern
- Implement Netlink confirmation mechanism for bundle sending
- Add comprehensive error handling and logging
- Restructure daemon architecture for better maintainability

Signed-off-by: Sylvain Pierrot <[email protected]>
@sylvain-pierrot sylvain-pierrot changed the title refactor(daemon,bp-socket): harden memory safety, error handling, and… feat: Complete ION integration with daemon refactoring and thread management Aug 29, 2025
@sylvain-pierrot sylvain-pierrot changed the title feat: Complete ION integration with daemon refactoring and thread management feat: Complete ION integration + daemon refactoring and thread management Aug 29, 2025
@sylvain-pierrot sylvain-pierrot merged commit 681a49b into main Aug 29, 2025
2 checks passed
@sylvain-pierrot sylvain-pierrot deleted the refactor/daemon-bp-socket-hardening branch August 29, 2025 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants