Skip to content

Commit d4a36a1

Browse files
committed
Fix async await anti-pattern causing lint error
1 parent 306e344 commit d4a36a1

File tree

1 file changed

+17
-20
lines changed

1 file changed

+17
-20
lines changed

ui/shared/auth/AuthService.js

Lines changed: 17 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -12,29 +12,26 @@ export default class AuthService {
1212
this.setUser = setUser;
1313
}
1414

15-
_fetchUser(userSession) {
15+
async _fetchUser(userSession) {
1616
const loginUrl = getApiUrl('/auth/login/');
1717

18-
// eslint-disable-next-line no-async-promise-executor
19-
return new Promise(async (resolve, reject) => {
20-
const userResponse = await fetch(loginUrl, {
21-
headers: {
22-
Authorization: `Bearer ${userSession.accessToken}`,
23-
'Access-Token-Expires-At': userSession.accessTokenExpiresAt,
24-
'Id-Token': userSession.idToken,
25-
},
26-
method: 'GET',
27-
credentials: 'same-origin',
28-
});
29-
30-
const user = await userResponse.json();
31-
32-
if (!userResponse.ok) {
33-
reject(new Error(user.detail || userResponse.statusText));
34-
}
35-
36-
resolve(new UserModel(user));
18+
const userResponse = await fetch(loginUrl, {
19+
headers: {
20+
Authorization: `Bearer ${userSession.accessToken}`,
21+
'Access-Token-Expires-At': userSession.accessTokenExpiresAt,
22+
'Id-Token': userSession.idToken,
23+
},
24+
method: 'GET',
25+
credentials: 'same-origin',
3726
});
27+
28+
const user = await userResponse.json();
29+
30+
if (!userResponse.ok) {
31+
throw new Error(user.detail || userResponse.statusText);
32+
}
33+
34+
return new UserModel(user);
3835
}
3936

4037
_clearRenewalTimer() {

0 commit comments

Comments
 (0)