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

Should not append /ipfs/id if the multiaddr already has it #730

Closed
daviddias opened this issue Jan 27, 2017 · 1 comment
Closed

Should not append /ipfs/id if the multiaddr already has it #730

daviddias opened this issue Jan 27, 2017 · 1 comment
Labels
kind/bug A bug in existing code (including security flaws)

Comments

@daviddias
Copy link
Member

@victorbjelkholm discovered it.

Bug here: https://github.com/ipfs/js-ipfs/blob/master/src/core/components/id.js#L21-L24
@daviddias daviddias added the kind/bug A bug in existing code (including security flaws) label Jan 27, 2017
@victorb
Copy link
Member

victorb commented Jan 28, 2017

Fixed by 718394a

@victorb victorb closed this as completed Jan 28, 2017
@awaisab172 awaisab172 mentioned this issue Jan 26, 2021
Open
@rtpro rtpro mentioned this issue Jan 27, 2021
Open
@awaisab172 awaisab172 mentioned this issue Jan 27, 2021
Open
@mkg20001 mkg20001 mentioned this issue Jan 27, 2021
Open
@snyk-bot snyk-bot mentioned this issue Feb 24, 2022
Open
@zi3lak zi3lak mentioned this issue Apr 27, 2022
Open
This was referenced Oct 31, 2022
Open
Open
@zi3lak zi3lak mentioned this issue Dec 26, 2022
Open
@zi3lak zi3lak mentioned this issue Jan 10, 2023
Open
@zi3lak zi3lak mentioned this issue Jul 8, 2023
Open
@zi3lak zi3lak mentioned this issue Dec 6, 2023
Open
@zi3lak zi3lak mentioned this issue Mar 22, 2024
Open
@zi3lak zi3lak mentioned this issue Apr 30, 2024
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
kind/bug A bug in existing code (including security flaws)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@victorb @daviddias