github/hedgedoc
1
0
Fork 0
mirror of https://github.com/hedgedoc/hedgedoc.git synced 2024-10-20 06:02:09 -04:00
Code Issues Projects Releases Packages Wiki Activity

auth: Remove userName parameter of removeToken function

Browse source 
As suggested by @innaytool

Signed-off-by: Philip Molares <philip.molares@udo.edu>
This commit is contained in:
Philip Molares 2021-01-23 22:24:59 +01:00
parent f68caab6e8
commit cc2fcac532
3 changed files with 4 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/api/private/tokens/tokens.controller.ts
View file

@ -48,7 +48,6 @@ export class TokensController {
@Delete('/:keyId') @Delete('/:keyId')
@HttpCode(204) @HttpCode(204)
async deleteToken(@Param('keyId') keyId: string) { async deleteToken(@Param('keyId') keyId: string) {
// ToDo: Get real userName return this.authService.removeToken(keyId);
return this.authService.removeToken('hardcoded', keyId);
} }
} }

2
src/auth/auth.service.spec.ts
View file

@ -152,7 +152,7 @@ describe('AuthService', () => {
describe('removeToken', () => { describe('removeToken', () => {
it('works', async () => { it('works', async () => {
await service.removeToken(user.userName, authToken.keyId); await service.removeToken(authToken.keyId);
}); });
}); });

5
src/auth/auth.service.ts
View file

@ -141,10 +141,9 @@ export class AuthService {
return user.authTokens; return user.authTokens;
} }
async removeToken(userName: string, keyId: string) { async removeToken(keyId: string) {
const user = await this.usersService.getUserByUsername(userName);
const token = await this.authTokenRepository.findOne({ const token = await this.authTokenRepository.findOne({
where: { keyId: keyId, user: user }, where: { keyId: keyId },
}); });
await this.authTokenRepository.remove(token); await this.authTokenRepository.remove(token);
} }