Authenticate with service not always respecting correct method

[chasefleming]

Problem

When passing a service into authenticate from the Discovery API, if the service method is "POP/RPC" or "TAB/RPC" it will still use "IFRAME/RPC" since that is the default config method. See the code below:

const isServiceMethodUnchangable = method =>
  ["EXT/RPC", "WC/RPC"].includes(method)

const discoveryWalletMethod = await config.first([
    "discovery.wallet.method",
    "discovery.wallet.method.default",
  ])

const method = isServiceMethodUnchangable(service?.method)
    ? service.method
    : discoveryWalletMethod || service.method