-
-
Notifications
You must be signed in to change notification settings - Fork 347
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
Fix check allowed for user #19
Conversation
Yeah, looks like a bug! Would be great to have a breaking test case for this! |
Test fail because the commit to coverall fails, we can ignore the status of |
It breaks the tutorial last step and all core functionality in this library. By the way, we now just start to RND this tool and wanna try to use it in our infrastructure systems, thanks :) |
Hey, that's great - the PR will definitely be merged. A failing test case will ensure that:
If you need help with writing the test, let me know! |
@arekkas yes, i need help, because i just start to learn and understand GO, my profile is python |
Ok, no problem. No telegram account, we'll use GitHub. I'll point you in the right directions tomorrow. |
Thank you for your contribution, I had a bit of spare time and quickly wrote a failing case for this and incorporated your PR. Cheers :) |
Thanks, i'll be waiting for new release 👍 |
There was an error on line #80 in warden_local.go. If it return from function on this line, we do not check roles allow/deny for requested policy