Authenticated as class decorator bug

[d0whc3r]

Informations

Type	Version
Bug	4.32.0

---

Description

@Authenticated decorator in class decorator doesn't get options

Acceptance criterias

import { Authenticated, Controller, Get, RouteService } from '@tsed/common';

@Controller('/')
@Authenticated('somestring')
export class RestController {

  constructor(private routeService: RouteService) {
  }

  @Get('/')
  public getRoutes(): any {
    return this.routeService.getAll();
  }
}

@OverrideMiddleware(AuthenticatedMiddleware)
export class AuthMiddleware implements IMiddleware {
  private JWT_SECRET = Buffer.from(process.env.JWT_SECRET || '', 'base64');

  constructor() {
  }
  use(
    @EndpointInfo() endpoint: EndpointMetadata,
    @Request() request,
    @Response() response,
    @Next() next: Express.NextFunction) {

    // endpoint is undefined
  }
}

How could I get options if endpoint is undefined?
endpoint.store.get(AuthenticatedMiddleware) doesn't work

[Romakita]

Hello,
Is not possible to use authenticated on class right know because authenticed decorator require option given to the decorator for the specific endpoint.
Isn’t a bug but an enhancement ;)

[d0whc3r]

Well, @Authenticated decorator works well as class decorator, but it can't accept "options"

[Romakita]

Hum yes, because, @EndpointInfo() endpoint: EndpointMetadata is the method to retrieve option given to authenticated decorator.

The hack right now, should be something like that:

@Controller('/')
export class RestController {

  @Authenticated('somestring')
  @All('/')
  all() {} // catch all requests

  @Get('/')
  public getRoutes(): any {
    return this.routeService.getAll();
  }
}

[Romakita]

🎉 This issue has been resolved in version 5.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀