Skip to content

CDK: Authenticator doesn't support auth with query params #3468

Closed
@keu

Description

@keu

Tell us about the problem you're trying to solve

The current implementation naively assumes that all authentication works through request headers, unfortunately, that is not the case, API keys can be sent via:

  • query params
  • cookies
  • body post

Describe the solution you’d like

The authenticator should support only one method:

def authenticate(self, session: Session):
    pass

Describe the alternative you’ve considered or used

Set Authenticator to NoAuth and implement authentication in request_params method.

Additional context

Add any other context or screenshots about the feature request here.

Are you willing to submit a PR?

Your answer

┆Issue is synchronized with this Asana task by Unito

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions