Reconsider use of pipe

[dapplion]

pipe is shown to be an expensive abstraction. In this specific use case it looks like a regular for loop would be enough.

js-libp2p-peer-store/src/address-book.ts

Lines 348 to 366 in 6d7b32c

	async function filterMultiaddrs (peerId: PeerId, multiaddrs: Multiaddr[], addressFilter: AddressFilter, isCertified: boolean = false) {
	return await pipe(
	multiaddrs,
	(source) => each(source, (multiaddr) => {
	if (!isMultiaddr(multiaddr)) {
	log.error('multiaddr must be an instance of Multiaddr')
	throw errcode(new Error('multiaddr must be an instance of Multiaddr'), codes.ERR_INVALID_PARAMETERS)
	}
	}),
	(source) => filter(source, async (multiaddr) => await addressFilter(peerId, multiaddr)),
	(source) => map(source, (multiaddr) => {
	return {
	multiaddr,
	isCertified
	}
	}),
	async (source) => await all(source)
	)
	}

[p-shahi]

Keeping this as low priority for now

[github-actions]

🎉 This issue has been resolved in version 6.0.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀