diff --git a/app/controllers/admin_controller.rb b/app/controllers/admin_controller.rb index 5926b40..d7a29c7 100755 --- a/app/controllers/admin_controller.rb +++ b/app/controllers/admin_controller.rb @@ -1,6 +1,6 @@ class AdminController < ApplicationController - # before_filter :administrative + before_filter :administrative, :if => :admin_param skip_before_filter :has_info def dashboard @@ -45,4 +45,10 @@ class AdminController < ApplicationController end end + + private + + def admin_param + params[:admin_id] != '1' + end end diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index a0ef5cf..9d5628d 100755 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -23,7 +23,7 @@ class ApplicationController < ActionController::Base def administrative if not is_admin? - reset_session + #reset_session redirect_to root_url end end diff --git a/app/controllers/tutorials_controller.rb b/app/controllers/tutorials_controller.rb index fb4d61d..0b88a9e 100755 --- a/app/controllers/tutorials_controller.rb +++ b/app/controllers/tutorials_controller.rb @@ -64,6 +64,9 @@ class TutorialsController < ApplicationController def insecure_components end + + def access_control + end def crypto end diff --git a/app/views/layouts/tutorial/_sidebar.html.erb b/app/views/layouts/tutorial/_sidebar.html.erb index 10e885f..acfdfd7 100755 --- a/app/views/layouts/tutorial/_sidebar.html.erb +++ b/app/views/layouts/tutorial/_sidebar.html.erb @@ -57,12 +57,12 @@ A6 Exposure <% end %> -
  • - <%= link_to url_access_tutorials_path do %> +
  • + <%= link_to access_control_tutorials_path do %>
    - A7 Access + A7 Access Control <% end %>
  • @@ -73,7 +73,7 @@ A8 CSRF <% end %>
    • -
  • +
  • <%= link_to insecure_components_tutorials_path do %>
    diff --git a/app/views/layouts/tutorial/access_control/_access_control_first.html.erb b/app/views/layouts/tutorial/access_control/_access_control_first.html.erb new file mode 100644 index 0000000..cce74f4 --- /dev/null +++ b/app/views/layouts/tutorial/access_control/_access_control_first.html.erb @@ -0,0 +1,107 @@ +
    +
    +
    + A7 - Missing Function Level Access Control +
    +
    +
    +
    +
    +
    + + + + Description + +
    +
    +
    + Many web applications check URL access rights before rendering protected links and buttons. However, applications need to perform similar access control checks each time these pages are accessed, or attackers will be able to forge URLs to access these hidden pages anyway. +
    +
    +
    +
    +
    + + + + Bug + +
    +
    +
    +

    + Rails provides the ability to apply before_filter(s) which run prior to rendering content to the user. This is helpful when restricting access to content based on the user's role. Currently, the methods to apply a before_filter already exist in the application controller but were forgotten when creating the administrative functionality. Notice an asbsence of the before_filter within app/controllers/admin_controller.rb +

    + 
    +        <%= %q{
+        class AdminController < ApplicationController
+        
+          skip_before_filter :has_info
+        } %>
+
    + +
    +
    +
    +
    +
    + + + + Solution + +
    +
    +
    +

    Failure to Restrict URL Access - ATTACK

    +

    + Request the following URL /admin/1/dashboard and have fun :-) +

    +

    Failure to Restrict URL Access - SOLUTION

    +

    + The code is already available to restrict access to the admin controller by role within app/controllers/application_controller.rb: +

    + 
    +        helper_method :current_user, :is_admin?
+        
+        def is_admin?
+            current_user.admin if current_user 
+          end
+      
+          def administrative
+            if not is_admin?
+             reset_session
+             redirect_to root_url
+           end
+          end
+
    +

    + Then add the following line within app/controllers/admin_controller.rb +

    + 
    +        class AdminController < ApplicationController
+
+          before_filter :administrative
+          skip_before_filter :has_info
+
    +
    +
    +
    +
    +
    + + + + Hint + +
    +
    +
    + I bet there is some admin functionality in here :-) +
    +
    +
    +
    +
    +
    \ No newline at end of file diff --git a/app/views/tutorials/access_control.html.erb b/app/views/tutorials/access_control.html.erb new file mode 100644 index 0000000..6796302 --- /dev/null +++ b/app/views/tutorials/access_control.html.erb @@ -0,0 +1,17 @@ +
    +
    +
    +
    + <%= render :partial => "layouts/tutorial/access_control/access_control_first" %> +
    +
    +
    +
    + + \ No newline at end of file diff --git a/config/routes.rb b/config/routes.rb index 39206f2..800d3d7 100755 --- a/config/routes.rb +++ b/config/routes.rb @@ -48,6 +48,7 @@ Railsgoat::Application.routes.draw do get "exposure" get "url_access" get "insecure_components" + get "access_control" get "ssl_tls" get "redirects" get "guard"