Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

Implementation of the request-response protocol #29

Merged
merged 37 commits into from
May 10, 2024
Merged

Implementation of the request-response protocol #29

merged 37 commits into from
May 10, 2024

Conversation

thewoodfish
Copy link
Member

  • Implementation of the request-response protocol and the high level interface to interact with it. This is exposed to the application by calling functions on the network core.
  • "Age of Empire", a program to demonstrate the capabilities of the library was also developed (still WIP).

thewoodfish and others added 30 commits May 3, 2024 16:40
@thewoodfish
chore: complete request_reponse impementation
ad49b52
@thewoodfish
chore: complete request_response impl
cd9449f
@thewoodfish
fix: impl of the request-response protocol
c9342b5
@thewoodfish
fix: added appropriate network stream interface to communicate with t…
6179ed6 
…he network layer from the application's event handler
@thewoodfish
add: added more functionality to age_of_empire
29a35f4
@thewoodfish
fix: added 'await' to all the futures of the application event handle…
fc15383 
…r functions
@thewoodfish
fix: added base58 decoding of a peer id string
018445d
@thewoodfish
chore: bebug request-response
a37cf46
@thewoodfish
chore: clean up request-response
6cf0e95
@thewoodfish
add: implementation of request-response
6fbaf78
@sacha-l
add: testing for network_id
3c16563
@thewoodfish @sacha-l
fix: made with_network_id panic if an invalid protocol_id passed in
bc7f527
@sacha-l
add: tests for valid network_id checks
c9f9c7a
@sacha-l
fix: add panic macro for tests
e8e589d
@sacha-l
add tests: core builder setup
836b941
@sacha-l
fix: async-std as dependency
7fdc895
@sacha-l
fix: add default values to prelude
25ad59e
@sacha-l
test: invalid keypair
4b7fd2c
@sacha-l
add: keypair protobuf tests
6b677b0
@sacha-l
fix: generate rsa on the fly for tests
ab2a93e
@sacha-l
chore: run fmt on all codebase
aa076ff
@sacha-l
fix merge conflicts: add tests to new core mod
b23d8a6
@sacha-l
fix: make tests compile
cf5888e
@sacha-l
fix: re-add panic for network_id validity
29770cf
@sacha-l
fix: update save_keypair_offline to handle non-existant file
60a04cb
@sacha-l
fix: tests for tokio
656dfc9
@sacha-l
fix Cargo issues
41d83de
@sacha-l
add: testing for network_id
d40b452
@thewoodfish @sacha-l
fix: made with_network_id panic if an invalid protocol_id passed in
4a6c28b
@sacha-l
add: tests for valid network_id checks
cc927cc
@sacha-l sacha-l merged commit b65446e into main May 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@thewoodfish @sacha-l