Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

fix: still load dag-pb, dag-cbor and raw when specifying custom formats #3132

Merged
merged 1 commit into from
Jul 1, 2020

Conversation

achingbrain
Copy link
Member

If we specify a formats array as part of the ipld options in in-proc nodes, it replaces the default list of dag-pb, dag-cbor and raw.

This change allows the loadFormat function to still resolve those formats even if the user has passed a format array that does not contain them.

Fixes #3129

If we specify a `formats` array as part of the ipld options in in-proc
nodes, it replaces the default list of dag-pb, dag-cbor and raw.

This change allows the `loadFormat` function to still resolve those
formats even if the user has passed a `format` array that does not
contain them.

Fixes #3129
@achingbrain achingbrain requested a review from hugomrdias June 29, 2020 15:22
@achingbrain achingbrain merged commit a96e3bc into master Jul 1, 2020
@achingbrain achingbrain deleted the fix/do-not-overwrite-ipld-formats branch July 1, 2020 09:54
@oed
Copy link
Contributor

oed commented Jul 1, 2020

Thanks @achingbrain :)

SgtPooki referenced this pull request in ipfs/js-kubo-rpc-client Aug 18, 2022
…ts (#3132)

If we specify a `formats` array as part of the ipld options in in-proc
nodes, it replaces the default list of dag-pb, dag-cbor and raw.

This change allows the `loadFormat` function to still resolve those
formats even if the user has passed a `format` array that does not
contain them.

Fixes #3129
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IPLD formats are overwritten in nodejs
2 participants