Support Dynamic Config

[harlan-zw]

Sometimes we won't know ahead of time the exact config to use for an oauth request, it would be nice if we could provide config as a function that takes the event property.

It's related to this issue: #48.

Bit edge case but I think it's a fairly simple change that provides much more flexibility.

config(event) {
  // do logic   
},

[atinux]

What you can do in the meantime is to create an event handler that return the event handler:

export default eventHandler(async event => {
  // get your config ...
  const ghConfig = await getGHConfig(event)
  return oauth.githubEventHandler({
     config: ghConfig,
     onSuccess(event, result) {

     }
  })(event)
})

[kbarnesMCC]

Just chiming in; I'm also going to be using this workaround as I have a need for dynamic config.

FWIW, if/when this supports gets implemented directly could consider rephrasing the handleMissingConfiguration error message to indicate it's not necessarily the NUXT_OAUTH... variables that aren't set; rather the config options haven't been passed.

[atinux]

Please checkout #277 (comment) for also another example

[atinux]

Just chiming in; I'm also going to be using this workaround as I have a need for dynamic config.

FWIW, if/when this supports gets implemented directly could consider rephrasing the handleMissingConfiguration error message to indicate it's not necessarily the NUXT_OAUTH... variables that aren't set; rather the config options haven't been passed.

Pull request is welcome to improve our error handling 💚