-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Create ipfs.add flag for specifying base of produced CID #5233
Comments
Actually what I plan to implement is a little different. It is a global |
Note that it doesn't imply CIDv1 because it makes sense to decouple the base used to display CIDv1 and the CID version of newly added files. |
@kevina Global flag for default base of outputted CIDs across entire app is a good idea, IIUC it would mitigate issue raised in #5234 👍 Note that it does not remove the need for opt-in Users should have an option to change the default base via ad-hoc command to enable batch scripting that outputs different base depending on added content. I see some scenarios where hashes for some files need to be as short as possible, while others need to conform to some standard or limitation, eg. be URL and Origin-safe. Having |
By a global flag I mean a command line flags that can be used with all |
Ack, finally got it, if it is a global commandline param, it brings much more value.
👍 from me, as long we implement the same in js-ipfs (cc @alanshaw) |
(subscribing) |
This is a small, but important piece on the road to CIDv1 in Base32: https://github.com/ipfs/ipfs/issues/337
IMO it should happen before anything else:
Note: it is a separate thing from "specifying full default cidv1 parameters", which is tracked in #5230
cc @kevina @kyledrake @alanshaw (for parity with js-ipfs – ipfs/js-ipfs#1440)
The text was updated successfully, but these errors were encountered: