Skip to content

Add authorization check#71

Merged
jboursier-mwb merged 2 commits intomainfrom
check-unauthorized
Apr 20, 2023
Merged

Add authorization check#71
jboursier-mwb merged 2 commits intomainfrom
check-unauthorized

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Mar 24, 2023

This PR adds a check for wrong credentials and simplifies the error handling on responses.

@ghost ghost force-pushed the check-unauthorized branch from 3180a45 to 0c54022 Compare March 24, 2023 07:05
@ghost ghost requested a review from jboursier-mwb March 24, 2023 07:05
@ghost ghost added the enhancement New feature or request label Mar 24, 2023
Security & Stuff added 2 commits March 24, 2023 07:24
Signed-off-by: Security & Stuff <74931194+SecurityAndStuff@users.noreply.github.com>
Signed-off-by: Security & Stuff <74931194+SecurityAndStuff@users.noreply.github.com>
@ghost ghost force-pushed the check-unauthorized branch from 5fef90e to ecbb5ce Compare March 24, 2023 07:24
Copy link
Copy Markdown
Collaborator

@jboursier-mwb jboursier-mwb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice improvement 🥳

@jboursier-mwb jboursier-mwb assigned ghost Mar 24, 2023
@jboursier-mwb jboursier-mwb added this pull request to the merge queue Apr 19, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Apr 19, 2023
@jboursier-mwb jboursier-mwb merged commit e8c79eb into main Apr 20, 2023
@ghost ghost deleted the check-unauthorized branch April 24, 2023 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant