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

replace particular request not found errors with public error #188

Merged
merged 1 commit into from
Aug 5, 2021

Conversation

dirkmc
Copy link
Collaborator

@dirkmc dirkmc commented Aug 5, 2021

Allows the caller to ignore a "request not found" error, for example when shutting down graphsync

graphsync.go Show resolved Hide resolved
@hannahhoward hannahhoward merged commit 9f51ecb into release/0.6.x Aug 5, 2021
@dirkmc dirkmc deleted the refactor/req-not-found-err branch August 5, 2021 12:44
hannahhoward added a commit that referenced this pull request Aug 5, 2021
* feat: fire network error when network disconnects during request (#164)

* release: 0.6.1

* Better logging for Graphsync traversal (#167)

* log gs traversal

* Apply suggestions from code review

Co-authored-by: dirkmc <[email protected]>

* add debug logs

* add debug logs

Co-authored-by: dirkmc <[email protected]>

* release: v0.6.2 (#168)

* Fix/log blockstore reads (#169)

* log gs traversal

* Apply suggestions from code review

Co-authored-by: dirkmc <[email protected]>

* add debug logs

* fixed logging

* Apply suggestions from code review

Co-authored-by: dirkmc <[email protected]>

* fixed error

Co-authored-by: dirkmc <[email protected]>

* release: v0.6.3 (#170)

* request queued hook

* test request queued hook

* release: v0.6.4

* Resolve 175 race condition, no change to hook timing (#178)

* feat(requestmanager): add request timing (#181)

* Add cancel request and wait function (#185)

* fix(responsemanager): fix error codes (#182)

enforce much tighter consistency with graphsync spec in usage of error codes

* refactor: replace particular request not found errors with public error (#188)

* release: 0.6.8

Co-authored-by: dirkmc <[email protected]>
Co-authored-by: Aarsh Shah <[email protected]>
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.

3 participants