Skip to content

Add request validators#4

Open
kalroy wants to merge 11 commits into
masterfrom
add_request_validators
Open

Add request validators#4
kalroy wants to merge 11 commits into
masterfrom
add_request_validators

Conversation

@kalroy
Copy link
Copy Markdown
Collaborator

@kalroy kalroy commented Nov 15, 2021

Overview

What this PR does / why we need it

These changes are required to validate authentication key before we hit the DB.
The authentication key is generated using base32 encoding so we should validate the key.

Special notes for your reviewer

Does this PR introduce a user-facing change?

The invalid keys generate an error Database Error.
There will be a change:

  • In case of valid key but not found in DB, the error will show Auth Request ID not found
  • In case of invalid key, the error will show Invalid auth code passed

Signed-off-by: Kallol Roy <karoy@progress.com>
Removing Sqllite3
Signed-off-by: Kallol Roy <karoy@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
Signed-off-by: Kallol Roy <karoy@progress.com>
atultherajput and others added 2 commits April 4, 2022 11:52
Signed-off-by: Atul Krishna <Atul.Krishna@progress.com>
Added HSTS response header.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants