-
Notifications
You must be signed in to change notification settings - Fork 25
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
Keploy Integration - Added sample Java application using JWT Token #76
base: main
Are you sure you want to change the base?
Conversation
Please review @Swpn0neel @charankamarapu |
Hi @charankamarapu Thanks for your suggestion. Have updated the PR to have token in header instead of body. |
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.
Please address the comments
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.
Please address the comments and before requesting for a review please follow the read me and add the record and test screenshots for native and docker case.
- Login | ||
|
||
```bash | ||
curl --location --request POST 'http://localhost:8080/users/login' \ |
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.
why is the native case removed ..?
These should be the headings
Prerequisites
Getting Started
Native Usage
Docker Usage
- here only you can write the command to build application image etc.
we dont need API Endpoints/ Running with Docker section
Hi @yaten2302, did you check it? |
Apologies for the delay, @jaiakash. I've DM you on Slack. |
Signed-off-by: Akash Jaiswal <[email protected]>
Signed-off-by: Akash Jaiswal <[email protected]>
Signed-off-by: Akash Jaiswal <[email protected]>
Signed-off-by: Akash Jaiswal <[email protected]>
Signed-off-by: Akash Jaiswal <[email protected]>
Signed-off-by: Pranshu Srivastava <[email protected]> Signed-off-by: Akash Jaiswal <[email protected]>
* feat:Add spring-mysql-redis to samples java Signed-off-by: Hermione Dadheech <[email protected]> * remove .idea and target folder Signed-off-by: Hermione Dadheech <[email protected]> * add .idea and target to gitignore Signed-off-by: Hermione Dadheech <[email protected]> * add .idea and target to gitignore Signed-off-by: Hermione Dadheech <[email protected]> * remove reports from .gitignore Signed-off-by: Hermione Dadheech <[email protected]> --------- Signed-off-by: Hermione Dadheech <[email protected]> Signed-off-by: Akash Jaiswal <[email protected]>
Signed-off-by: Akash Jaiswal <[email protected]>
Hi @PranshuSrivastava @Sonichigo @charankamarapu, I am incredibly sorry for the delay. Have updated the PR. Can you please suggest what changes are required in this readme file or maybe the overall project? Thanks cc @yaten2302 |
Pull Request Template
Description
Hi me (@jaiakash) and @amitpanwar789 have worked on the sample Java application with JWT token creation and verification. This sample helps to use the Keploy to test in JAVA.
Fixes keploy/keploy#2016
Type of change
Please delete options that are not relevant.
Checklist: