-
Notifications
You must be signed in to change notification settings - Fork 65
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
Write a Real World Example for Auth #4819
Comments
I would like to work on this. Kindly assign to me |
Hi @DevTomilola-OS! 🚀 We value your contributions and look forward to collaborating with you.
Remember, no contribution is too small, and your feedback is invaluable. Feel free to ask questions, propose ideas, or report issues. Together, we can make Ballerina even better! Happy coding! 🎉 |
Hello @keizer619 I'm sorry I won't be working on this issue anymore. Thank you. |
@keizer619 @ThisaruGuruge can you please assign this to me, I can work on this. |
Hi @harshalkh, Thank you for considering this issue. I have assigned it to you now. |
@ThisaruGuruge @DimuthuMadushan I am trying to setup few things and giving a try to execute APIs. Got some strange issue - /foo/bar works without any issue, whereas /accounts/account not... harshalkh/module-ballerina-auth#3 |
Hi @harshalkh, It's great to see you working on this. I went through your code and noticed that you forgot to enable SSL in the HTTP listener, which might be causing the issue. To fix this, you can set up the SSL certificate and key using the I also recommend adding Ballerina tests to make it easy to test your APIs. It's pretty straightforward, and it will make your example more complete and practical. To learn more about Ballerina testing, check out this short YouTube video series: Ballerina Testing Video Series |
@DimuthuMadushan as per suggestion I have enabled SSL as well as added test cases. Still not able to figure out where it's failing. Can you pls refer above links and suggest? time=2023-10-13T21:10:02.266Z level=DEBUG module=ballerina/auth message="File user store values are not provided or not properly configured." UPDATE- able to get it running now after adding Config.toml in test folder as well Thanks |
@DimuthuMadushan / @ThisaruGuruge Finally I am able to raise PR - ballerina-platform/module-ballerina-auth#1057 Please review and let me know if any suggestions. |
Hi @harshalkh That's great! I glanced at the PR and made a few minor suggestions. |
@DimuthuMadushan @ThisaruGuruge - I have made requested changes. Please verify and let me know if anything needs to change. |
@DimuthuMadushan @ThisaruGuruge - I have done requested changes and verified test cases as well.. please check. |
Hi @harshalkh we have approved your PR. Thank you for your valuable contribution. 🎉 |
Description:
Currently, there is an example in the Ballerina auth repository that utilizes the LDAP user store. We need to add another example to the auth module, preferably one that incorporates file user store with scopes.
The text was updated successfully, but these errors were encountered: