Allow "Addresses.Gateway" / "Addresses.API" to be array (Allow gateway/api to listen on multiple addresses)

[mkg20001]

Version information:

go-ipfs version: 0.4.17-
Repo version: 7
System version: amd64/linux
Golang version: go1.10.3

Type: feature

Description: Allow "Addresses.Gateway" / "Addresses.API" properties to be arrays (Allow gateway/api to listen on multiple addresses)

Currently it's only possible to make the gateway/API listen on one address only.

Instead both of these parameters should accept arrays to make it possible to for ex listen on /ip6/::1/tcp/8080 as well.

[kjzz]

hey @Stebalien,if anyone work for this issue?if not,can i do for this?

[overbool]

@kjzz sorry, I'm fixing it.

[overbool]

@Stebalien this feature refers to go-ipfs-config, should modify it first?

[ghost]

this feature refers to go-ipfs-config, should modify it first?

Yes, and there's an fs-repo migration required, to do the string-to-array conversion in existing config files: https://github.com/ipfs/fs-repo-migrations -- best to coordinate with @kevina here

[Stebalien]

To repeat my comment from ipfs/go-ipfs-config#11 (comment), if we're not careful, we'll break ipfs config "Addresses.API" "/ip4/127.0.0.1/tcp/1234". Unfortunately, this will break a ton of automation.

Instead of migrating, I'd prefer to support both through something like ipfs/go-ipfs-config#11. However, supporting two JSON config formats is a bit hacky.

[ghost]

Taken care of in #5532 and #5553