New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix 500 code for authentication issue #103

Open

aPavar wants to merge 4 commits into master from fix/500-authentication

aPavar commented Nov 15, 2023 •

edited by EvgenySafronov

Loading

PR Details

There was created AqualityAuthenticationException class with 401 status code
There was changing lombok from 1.18.24 to 1.18.30

Related Issue

aquality-automation/aquality-tracking#141

How Has This Been Tested

Local tests pass.

Test caces:

user doesn't exists - 401 - ok
user exists, password incorrect - 401 - ok
user exists, password correct - authorization success - ok

Checklist

My code follows the code style of this project.
My change requires a change to the documentation.
I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I have added tests to cover my changes.
All new and existing tests passed.

aPavar added 4 commits

October 29, 2023 14:25


          Add case for AqualityAuthenticationException

81a2dd8


          change version in pom


          update changelog.md

3bd42c2


          update version lombok for running tests locally

0edcef9

aPavar requested a review from kknqqq

November 15, 2023 13:55

kknqqq approved these changes

View reviewed changes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet