Skip to content

Commit c053cb3

Browse files
authored
fix: state should not be required on OAuth Response (#4995)
1 parent 2dd9136 commit c053cb3

File tree

1 file changed

+2
-2
lines changed
  • packages/@ionic/cli/src/lib/oauth

1 file changed

+2
-2
lines changed

packages/@ionic/cli/src/lib/oauth/oauth.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,15 +117,15 @@ export abstract class OAuth2Flow<T extends OpenIdToken> {
117117
if (req.url) {
118118
const params = qs.parse(req.url.substring(req.url.indexOf('?') + 1));
119119

120-
if (params.code && params.state) {
120+
if (params.code) {
121121
res.writeHead(200, { 'Content-Type': ContentType.HTML });
122122
res.end(successHtml);
123123
req.socket.destroy();
124124
server.close();
125125

126126
const authResult = {
127127
code: Array.isArray(params.code) ? params.code[0] : params.code,
128-
state: Array.isArray(params.state) ? decodeURI(params.state[0]) : decodeURI(params.state),
128+
state: params.state ? (Array.isArray(params.state) ? decodeURI(params.state[0]) : decodeURI(params.state)) : '',
129129
};
130130

131131
resolve(authResult);

0 commit comments

Comments
 (0)