openhub.net
Black Duck Software, Inc.
Black Duck Open Hub
Follow @
OH
Sign In
Join Now
Projects
People
Organizations
Tools
Blog
BDSA
Projects
People
Projects
Organizations
Forums
C
catalyst-app-tutorial-kiffin-authissues
Settings
|
Report Duplicate
0
I Use This!
×
Login Required
Log in to Open Hub
Remember Me
Inactive
Commits
: Listings
Analyzed
about 15 hours
ago. based on code collected
about 20 hours
ago.
Apr 19, 2023 — Apr 19, 2024
Showing page 1 of 1
Search / Filter on:
Commit Message
Contributor
Files Modified
Lines Added
Lines Removed
Code Location
Date
Aha, the thing which retrieves a list of roles really really needs to be called 'roles' - this also fixes the bug in the template display, so I've put that back. Note that Plugin::Authorization::Roles is rubbish, it should at least spit warnings that the user hasn't got a roles method rather than exploding, and I actutally think it should be turned inside out as there is a lot of badly factored code in there, and those 'check_' methods are just abominable - throwing away all exceptions ever IS NOT A GOOD STRATEGY. Note that the TestApp still explodes, but not for the original reasons, and my tests now pass. :)
t0m
More...
almost 15 years ago
Tada, out pops our exception
t0m
More...
almost 15 years ago
Remove the peek stuff. Switch over to assert_user_roles, as check_user_roles unilaterally throws away all exceptions
t0m
More...
almost 15 years ago
Inline the plugin in question into our dist
t0m
More...
almost 15 years ago
Method finding magic. Devel::Peek will print the filename I should inline
t0m
More...
almost 15 years ago
No, I'm just an idiot, and I can't read the tutorial. This is what I need to test, and I can emulate the issue with the behavior seen..
t0m
More...
almost 15 years ago
And some tests. I never ever seem to get unauthorized, even for the users which aren't admin, so 2 tests fail currently
t0m
More...
almost 15 years ago
Add .gitignore file
t0m
More...
almost 15 years ago
I think that the code shown by the tutorial should include this, or perhaps the bug is we stopped stringifying to the role name??
t0m
More...
almost 15 years ago
Test login for all three users
t0m
More...
almost 15 years ago
Add trivial login test
t0m
More...
almost 15 years ago
Ewww, no tests, and the tests we do have fail
t0m
More...
almost 15 years ago
And more dependencies
t0m
More...
almost 15 years ago
Fix POD
t0m
More...
almost 15 years ago
Add description of the issue
t0m
More...
almost 15 years ago
Add Catalyst::Authentication::Realm::SimpleDB, another missing dependency
t0m
More...
almost 15 years ago
Remove pod coverage test
t0m
More...
almost 15 years ago
Add missing deps, 3 of which I don't have on my machine
t0m
More...
almost 15 years ago
Initial commit
t0m
More...
almost 15 years ago
This site uses cookies to give you the best possible experience. By using the site, you consent to our use of cookies. For more information, please see our
Privacy Policy
Agree