The additional checks were superfluous as it would only ever return
error if one of those protocols were set. Also: a returned error
*should* mean get out of there, without having to check more conditions.
Closes #14104
if(result)
break;
- /* result is only used when for ipfs and ipns, ignored otherwise */
result = url_proto(&per->this_url, config, &use_proto);
- if(result && (use_proto == proto_ipfs || use_proto == proto_ipns))
+ if(result)
break;
#ifndef DEBUGBUILD