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.

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

Send deal rejection reason to client #436

Merged
merged 1 commit into from
Apr 13, 2022
Merged

Send deal rejection reason to client #436

merged 1 commit into from
Apr 13, 2022

Conversation

dirkmc
Copy link
Contributor

@dirkmc dirkmc commented Apr 12, 2022

Instead of just closing the connection:

$ ./boost deal \
--provider=t01000 \
--commp=baga6ea4seaqhiabj4lnvg5nfgoe6xufk3qwdl54wq7qsu7fqrk3vu6axxukccmq \
--piece-size=2048 \
--car-size=1101 \
--payload-cid=bafk2bzacearocrcaflv5hesscsbiv64x4om5m64nsh347wedig3nn5l3anypi \
--verified=false \
--http-url=http://localhost:5000/1k-file.car
2022-04-12T11:46:35.278+0200	FATAL	boost	boost/main.go:41
	deal proposal rejected: server error: provider has insufficient funds to accept deal

Fixes #433

@dirkmc dirkmc force-pushed the fix/deal-reject-reason branch from 4ca5419 to a9d6929 Compare April 12, 2022 09:52
@dirkmc dirkmc merged commit 21b2c90 into main Apr 13, 2022
@dirkmc dirkmc deleted the fix/deal-reject-reason branch April 13, 2022 12:51
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.

Boost closes connection with no response when deal is rejected
2 participants