Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Validation of originator payloads (sid, timestamp, signature), with misbehavior reports #302

Open
Tracked by #118
richardhuaaa opened this issue Dec 3, 2024 · 1 comment
Open
Tracked by #118

Validation of originator payloads (sid, timestamp, signature), with misbehavior reports #302

richardhuaaa opened this issue Dec 3, 2024 · 1 comment
Assignees
@neekolas
Labels
sync Synchronization between nodes

Comments

@richardhuaaa
Copy link
Contributor

No description provided.

@richardhuaaa richardhuaaa mentioned this issue Dec 3, 2024
Closed
@richardhuaaa richardhuaaa self-assigned this Dec 3, 2024
@richardhuaaa richardhuaaa mentioned this issue Dec 6, 2024
Merged
richardhuaaa added a commit that referenced this issue Dec 19, 2024
@richardhuaaa
Validate node ID, sequence ID and timestamp on synced payloads (#311)
f223c36 
This adds initial validation on synced payloads.

Still to be done:

- Misbehavior reporting system
- Payload-specific validation

#302 

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

## Summary by CodeRabbit

- **New Features**
	- Enhanced query logic to ensure unique results for originator nodes.
- Added a method to access `OriginatorNs` value in the
`OriginatorEnvelope` struct.
- Improved stream management and envelope processing in the sync worker.

- **Bug Fixes**
- Refined error handling for stream connections and envelope validation.

- **Refactor**
- Restructured stream and envelope handling to improve data
encapsulation and processing logic.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
@neekolas
Copy link
Contributor

neekolas commented Jan 2, 2025

TODO @neekolas tackle some of the TODOs from #311

@richardhuaaa richardhuaaa mentioned this issue Jan 3, 2025
Merged
@neekolas neekolas added the sync Synchronization between nodes label Jan 3, 2025
@neekolas neekolas moved this to Todo in Decentralization Jan 3, 2025
@neekolas neekolas moved this from Todo to In Progress in Decentralization Jan 3, 2025
@mkysel mkysel mentioned this issue Feb 20, 2025
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@neekolas neekolas

Labels
sync Synchronization between nodes
Projects
Decentralization
Status: In Progress
Milestone
No milestone
Development

No branches or pull requests
2 participants
@neekolas @richardhuaaa