🚀 Feature: OAuth2 allow for additional query parameters

[codercatdev]

🔖 Feature description

When calling createOAuth2Session there should be a way to pass in additional parameters. An example would be when trying to call auth0, I want to be able to pass in prompt=login to forcibly have the user login every time they call login. Currently when calling the authorization endpoint Auth0 sees that their is a user already active and automatically logged in.

🎤 Pitch

I would like to change
createOAuth2Session(provider: string, success?: string, failure?: string, scopes?: string[]): void | URL;

in the sdk to take in a new parameter additionalQuery.

      await account.createOAuth2Session(
        "auth0",
        `${location.origin}/`,
        `${location.origin}/?login=failed`,
        []
      ); 

createOAuth2Session(provider: string, success?: string, failure?: string, scopes?: string[], scopes?: string[], additionalQueries?: string[]): void | URL;

This would require a change within Appwrite's internals here

appwrite/src/Appwrite/Auth/OAuth2.php

Line 41 in fb5805a

public function __construct(string $appId, string $appSecret, string $callback, array $state = [], array $scopes = [])

to include the

protected array $additionalQuery;
...
    public function __construct(string $appId, string $appSecret, string $callback, array $state = [], array $scopes = [], array $additionalQueries)
    {
        $this->appID = $appId;
        $this->appSecret = $appSecret;
        $this->callback = $callback;
        $this->state = $state;
        foreach ($scopes as $scope) {
            $this->addScope($scope);
        }
        foreach ($additionalQueries as $additionalQuery) {
            $this->addAdditionalQuery($additionalQuery);
        }
    }
...
    protected function addAdditionalQuery(string $query): OAuth2
    {
        // Add a query to the additionalQueries array if it isn't already present
        if (!\in_array($query, $this->additionalQueries)) {
            $this->additionalQueries[] = $query;
        }

        return $this;
    }
...

    /**
     * @return array
     */
    protected function getAdditionalQueries(): array
    {
        return $this->additionalQueries;
    }

Then you can make overrides like in Auth0

appwrite/src/Appwrite/Auth/OAuth2/Auth0.php

Line 43 in fb5805a

public function getLoginURL(): string

    public function getLoginURL(): string
    {
        return 'https://' . $this->getAuth0Domain() . '/authorize?' . \http_build_query([
            'client_id' => $this->appID,
            'redirect_uri' => $this->callback,
            'state' => \json_encode($this->state),
            'scope' => \implode(' ', $this->getScopes()),
            'response_type' => 'code'
            \implode(' ', $this->getAdditionalQueries()), //example `prompt=login`
        ]);
    }

👀 Have you spent some time to check if this issue has been raised before?

• I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

• I have read the Code of Conduct

[codercatdev]

Right now I am solving this by hitting my logout window directly for the next login, which is not ideal having to pop user over to new window.

  const deleteSessions = async () => {
    if (!account) return null;
    try {
      await account.deleteSessions();
      setUser(null);
      setSessions(null);
      const wnd = window.open("https://codingcatdev.us.auth0.com/v2/logout");
      setTimeout(() => {
        wnd?.close();
      }, 1000);
    } catch (error) {
      throw error;
    }
  };