Skip to content
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.

Sign up for GitHub

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

Jump to bottom

A7 MFLAC Documentation + Unit Test Issue #318

Open
nvisium-john-poulin opened this issue Feb 21, 2018 · 0 comments
Open

A7 MFLAC Documentation + Unit Test Issue #318

nvisium-john-poulin opened this issue Feb 21, 2018 · 0 comments
Labels
bug Documentation

Comments

@nvisium-john-poulin
Copy link
Contributor

The documentation (https://github.com/OWASP/railsgoat/wiki/A7-Missing-Function-Level-Access-Control--(Admin-Controller)) utilizes before_filter which has been deprecated in Rails 5.

That aside, the unit test still fails because it's expecting the path to be / not /dashboard/home:

     Failure/Error: expect(current_path).to eq("/")

       expected: "/"
            got: "/dashboard/home"
nvisium-john-poulin added a commit to nvisium-john-poulin/railsgoat that referenced this issue Feb 21, 2018
@nvisium-john-poulin
Updated unit test for url_access to have correct path, allowing users…
495df2c 
… to pass the test as part of OWASP#318
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nvisium-john-poulin