-
Notifications
You must be signed in to change notification settings - Fork 4
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
We need a break
!
#83
We need a break
!
#83
Conversation
} | ||
case PIN_HASH: { | ||
if (!ipfsHash) { | ||
log('PIN_HASH: no ipfsHash given: %O', message.data); | ||
if (log.enabled) { | ||
// Only stringify when absolutely necessary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add this to the REPLICATE
case below and the 'could not parse pinner message' error above?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh, yeah sure, I missed those!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
src/Pinion.ts
Outdated
} catch (caughtError) { | ||
if (log.enabled) { | ||
// Only stringify when absolutely necessary | ||
log('Could not parse pinner message: %O', message.data); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't even stringify when absolutely necessary 😛
EDIT: Nor does the REPLICATE
one
b4b6a2c
to
41a6716
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks sorted to me 👍
This PR adds a
break
statement for theANNOUNCE_CLIENT
case to not fall through to thePIN_HASH
case.Also we're sneaking in a commit to improve the buffer logging in debug mode