feat: Properly implement prompt

xhayper · Aug 7, 2022 · 90c024a · 90c024a
1 parent c239dba
commit 90c024a
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/examples/simple_oauth.js b/examples/simple_oauth.js
@@ -27,5 +27,6 @@ client.login({
         "messages.read",
         "rpc",
         "rpc.notifications.read"
-    ]
+    ],
+    prompt: "none" // Only prompt once
 }); // These are the allowed scopes for unapproved app.
diff --git a/examples/simple_oauth.ts b/examples/simple_oauth.ts
@@ -27,5 +27,6 @@ client.login({
         "messages.read",
         "rpc",
         "rpc.notifications.read"
-    ]
+    ],
+    prompt: "none" // Only prompt once
 }); // These are the allowed scopes for unapproved app.
diff --git a/src/Client.ts b/src/Client.ts
@@ -158,8 +158,7 @@ export class Client extends (EventEmitter as new () => TypedEmitter<ClientEvents
                 await this.fetch("POST", "/oauth2/token/rpc", {
                     data: new URLSearchParams({
                         client_id: this.clientId,
-                        client_secret: this.clientSecret ?? "",
-                        prompt: options.prompt ?? "consent"
+                        client_secret: this.clientSecret ?? ""
                     })
                 })
             ).data.rpc_token;
@@ -170,7 +169,8 @@ export class Client extends (EventEmitter as new () => TypedEmitter<ClientEvents
                 scopes: options.scopes,
                 client_id: this.clientId,
                 rpc_token: options.useRPCToken ? rpcToken : undefined,
-                redirect_uri: options.redirect_uri ?? undefined
+                redirect_uri: options.redirect_uri ?? undefined,
+                prompt: options.prompt ?? "consent"
             })
         ).data;