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

feat: Add bitswap server #27

Merged

oblique merged 24 commits into eigerco:main from fl0rek:feat/server

Mar 1, 2024

Member

fl0rek commented Feb 23, 2024 •

edited

Loading

closes #9

fl0rek force-pushed the feat/server branch 2 times, most recently from 9c6b7d5 to 14464e6 Compare

February 23, 2024 09:06


          feat: Add bitswap server

a4bdc2f

fl0rek force-pushed the feat/server branch from 14464e6 to a4bdc2f Compare

February 23, 2024 09:14

fl0rek added 2 commits

February 23, 2024 16:21


          Polish things up

2906ec6


          Add wishlist tests

23c6bed

fl0rek force-pushed the feat/server branch from a030533 to c7d7b5d Compare

February 26, 2024 11:01


          clippy

7751c28

fl0rek force-pushed the feat/server branch from c7d7b5d to 7751c28 Compare

February 26, 2024 11:01

fl0rek added 4 commits

February 26, 2024 12:03


          Update cargo-udeps

15aaee4

wip

a7b261c


          cargo lock

f6da917


          fixes

38ac701

fl0rek marked this pull request as ready for review

February 26, 2024 13:44

fl0rek requested review from oblique and zvolin

February 26, 2024 13:44

zvolin reviewed

View reviewed changes

src/client.rs Outdated Show resolved Hide resolved

zvolin reviewed

View reviewed changes

src/client.rs Show resolved Hide resolved

zvolin reviewed

View reviewed changes

src/server.rs Outdated Show resolved Hide resolved

zvolin reviewed

View reviewed changes

src/server.rs Outdated Show resolved Hide resolved

zvolin reviewed

View reviewed changes

src/server.rs Outdated Show resolved Hide resolved

zvolin reviewed

View reviewed changes

src/server.rs Outdated Show resolved Hide resolved

zvolin reviewed

View reviewed changes

src/server.rs Outdated Show resolved Hide resolved

zvolin reviewed

View reviewed changes

tests/bitswap.rs Show resolved Hide resolved

zvolin reviewed

View reviewed changes

Member

zvolin left a comment

naisu, left some ideas

zvolin reviewed

View reviewed changes

src/server.rs Outdated Show resolved Hide resolved

fl0rek and others added 4 commits

February 27, 2024 08:44


          Update src/client.rs

6885e0f

Co-authored-by: Maciej Zwoliński <[email protected]>


          Update src/server.rs

1fbf41e

Co-authored-by: Maciej Zwoliński <[email protected]>


          Update src/server.rs

1fb88e2

Co-authored-by: Maciej Zwoliński <[email protected]>


          review

a7a8f42

fl0rek added 3 commits

February 27, 2024 09:09


          Merge remote-tracking branch 'origin/feat/server' into feat/server

5ed93cd


          commiting because I've managed to crash the compiler

b696c25


          push

e7a2506

fl0rek force-pushed the feat/server branch from a2c8879 to e7a2506 Compare

February 27, 2024 12:28

oblique reviewed

View reviewed changes

src/client.rs Outdated Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

src/server.rs Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

src/server.rs Show resolved Hide resolved

fl0rek and others added 5 commits

February 27, 2024 16:41


          Update src/client.rs

b3eabbd

Co-authored-by: Yiannis Marangos <[email protected]>


          Yiannis review 0 update 0

ef465d6


          fixes

cc81ab8


          remove take

f35f420


          Arc<PeerId>

798d799

fl0rek force-pushed the feat/server branch from 1c452fe to 798d799 Compare

February 28, 2024 16:54


          test

d1b6b22

Member

zvolin commented Feb 28, 2024

closes #9

zvolin linked an issue

that may be closed by this pull request

Implement server #9

Closed


          Remove last remnants of cancellation

741e545

fl0rek requested review from zvolin and oblique

February 29, 2024 13:02

zvolin approved these changes

View reviewed changes

Member

zvolin left a comment

nice one, lgtm


          don't close stream after sending entire message

0ab9feb

oblique approved these changes

View reviewed changes

Member

oblique left a comment

Nice! Cool tests too.

examples/client.rs Outdated Show resolved Hide resolved

src/server.rs Outdated Show resolved Hide resolved

tests/utils/mod.rs Show resolved Hide resolved


          Update src/server.rs

b0280d7

Co-authored-by: Yiannis Marangos <[email protected]>

oblique merged commit 7736aab into eigerco:main

5 checks passed

zvolin mentioned this pull request

Bitswap server eigerco/lumina#196

Closed

This was referenced Apr 15, 2024

chore: release v0.1.0 #42

Closed

chore: release v0.1.0 #44

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet