-
Notifications
You must be signed in to change notification settings - Fork 532
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
Add API Auth: Basic authentication #610
Comments
Hi @ashitaprasad , I'd like to work on this issue want to add Basic authentication for the APIs. Could you please assign it to me? Let me know if there's anything specific you'd like me to keep in mind. Thanks! |
@Muhammad-Rebaal You can go through - Why we do not assign issues to anyone? You can through the updated application guide here to learn how you can share your ideas and get feedback. |
Ok @ashitaprasad ! I took a look at that and would try to solve this exciting issue as soon as possible. |
Hloo can i work in this |
Sure @Nithin2511 |
KINDLY REVIEW MY PULL REQUEST |
@Nithin2511 You PR has been reviewed. You need to go through the project, documentation and the CONTRIBUTING GUIDE first to understand more about the project. |
@ashitaprasad Can i also work in this or is it fixed already? |
@CinnamonRoll10 you can raise an issue. |
@ashitaprasad, I have thought of an approach. First, I will create a new BasicAuthModel class to store the username, password, and enabled state. Then, I will update the HttpRequestModel to include a basicAuth field and add methods to generate the Base64 encoded Authorization header. Next, I will implement a BasicAuthentication widget for user input of credentials. After that, I will integrate this widget into the request editor UI. (Maybe this thing needs to be done during the GSOC period) Finally, I will modify the request handling logic to automatically include the Authorization header when Basic Authentication is enabled. |
@ashitaprasad I’ve raised a PR to fix the Basic Auth issue.. #687 Please review it and merge if everything looks good. If any improvements are needed, just let me know, and I’ll update it. I’ve also added support for future authentication options, so it’s easier to extend in upcoming updates. Hope my proposal for GSoC gets accepted for issue #609! thanking you |
Tell us about the task you want to perform and are unable to do so because the feature is not available
Add Basic authentication support for APIs where a verified username and password is sent with API request
The text was updated successfully, but these errors were encountered: