Issue migrating to v2.0, audience parameter removed

[dsilva609]

Awesome work with getting v2.0 out to work with remix-auth v4. I am running into issues trying to migrate to the latest version though. I noticed that the audience parameter was removed from the constructor and when I tried to authenticate without it, I would still get an auth token but it was unusable for me. I went back to version 1.10 to see what would happen if I didn't specify an audience value and experienced the same issue. Was there a reason this was removed and is there any kind of workaround to get support for an audience? This is preventing me from migrating over to v2.0 at the moment.

[ciyer]

Yes, I think this was temporarily lost in the remix-auth-oauth2 project as well: sergiodxa/remix-auth-oauth2@65b147d

I am working around this by creating a subclass:

interface Auth0StrategyWithAudienceConstructorOptions
  extends Auth0Strategy.ConstructorOptions {
  /**
   * The audience of the token to request from the Identity Provider. This is
   * used when the Identity Provider requires a specific audience to be set on
   * the token.
   *
   * This can be a string or an array of strings.
   */
  audience?: string | string[];
}

class Auth0StrategyWithAudience<User> extends Auth0Strategy<User> {
  audience: string | string[];
  constructor(
    options: Auth0StrategyWithAudienceConstructorOptions,
    verify: ConstructorParameters<typeof Auth0Strategy<User>>[1]
  ) {
    super(options, verify);
    this.audience = options.audience ?? [];
  }

  protected authorizationParams(params: URLSearchParams, request: Request) {
    const newParams = super.authorizationParams(params, request);
    if (this.audience) {
      if (Array.isArray(this.audience)) {
        for (const audience of this.audience) {
          newParams.append("audience", audience);
        }
      } else newParams.append("audience", this.audience);
    }
    return newParams;
  }
}

But I agree, it would be good to fix in the main repo. @danestves I'm happy to contribute this if you want (though it's pretty easy to do, and it might be easier if you just implement it yourself).

[danestves]

Hi! I can work on that this weekend that I'm free 🙌