-
Notifications
You must be signed in to change notification settings - Fork 716
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.
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
refactor: litmus-portal authentication server #2867
refactor: litmus-portal authentication server #2867
Conversation
Signed-off-by: Hemanth Krishna <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
Signed-off-by: Hemanth Krishna <hkpdev008@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left few comments, PTAL
Signed-off-by: Hemanth Krishna <hkpdev008@gmail.com>
Hey @SarthakJain26 , Thank you for the comments. I have addressed/resolved the same. |
@ishangupta-ds @SarthakJain26 @rajdas98 All the requested changes have been addressed and resolved 😃 |
9cc7655
to
efe9c0e
Compare
Signed-off-by: Hemanth Krishna (DarthBenro008) <hkpdev008@gmail.com>
efe9c0e
to
02b2ab0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: DarthBenro008 <hkpdev008@gmail.com>
Signed-off-by: DarthBenro008 <hkpdev008@gmail.com>
@gdsoumya @rajdas98, the sample deployment of the codebase can be found at: https://litmusportal-production-frontend-service-darthbenro008.cloud.okteto.net Username: Strict policy check has been enabled for this deployment and can be tested by changing credentials. |
Signed-off-by: DarthBenro008 <hkpdev008@gmail.com>
Authentication Server Refactor
This PR comprehends a full re-write of litmus-portal's golang authentication server.
The server has been developed keeping in mind that the litmus-portal frontend does not need to make any change in request/response.
Routes Included
Dependencies Used
Results
Performance Gain
The current authentication server took 14 seconds for an average response
The re-written authentication server took 0.3 seconds for an average response
Easier Packaging of Code
The packages and folders have been completely simplified. Everything in the codebase has been decoupled for easier development and access.
Before:

After:

Try out
You can find this codebase deployed on https://litmusportal-production-frontend-service-darthbenro008.cloud.okteto.net/
Username:
admin
Password:
litmus
Alternatively, you can also deploy this codebase on your own
Okteto Cluster
by clicking hereToDo