bcms_cas 1.1.1 → 1.1.2

data/app/models/cms/temporary_user.rb

@@ -32,13 +32,6 @@ module Cms
       false
     end
 
-#    def cms_access?
-#      groups.each do |g|
-#        return true if g.cms_access?
-#      end
-#      false
-#    end
-
     # Returns all sections that this user has rights to view.
     #
     # Overrides core behavior of User to avoid including user as part of the query (since temp users doen't exist in the database).

data/lib/bcms_cas/authentication.rb

@@ -12,20 +12,20 @@ module Cas
     def self.included(controller_class)
       controller_class.send(:include, InstanceMethods)
 
+      # This reorders the filters applied to requests so that CAS authenticates a user first, before Authorization for documents or pages is checked.
       controller_class.skip_filter :check_access_to_page
+      controller_class.skip_filter :try_to_stream_file
+      
       controller_class.before_filter CASClient::Frameworks::Rails::GatewayFilter
       controller_class.before_filter :login_from_cas_ticket
-      controller_class.before_filter :check_access_to_page_normally
+      controller_class.before_filter :try_to_stream_file      
+      controller_class.before_filter :check_access_to_page
+      
     end
 
     # Each instance of the controller will gain these methods.
     module InstanceMethods
 
-      # This exists because we want to force this to happen AFTER login_from_cas_ticket. There may be a better way to do this.
-      def check_access_to_page_normally
-        check_access_to_page
-      end
-
       # Attempts to set the current user based on the session attribute set by CAS.
       def login_from_cas_ticket
         logger.debug "Checking for cas login. The current_user is '#{@current_user.login}'." if @current_user

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: bcms_cas
 version: !ruby/object:Gem::Version 
-  hash: 17
+  hash: 23
   prerelease: false
   segments: 
   - 1
   - 1
-  - 1
-  version: 1.1.1
+  - 2
+  version: 1.1.2
 platform: ruby
 authors: 
 - BrowserMedia
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-05 00:00:00 -04:00
+date: 2010-10-19 00:00:00 -04:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency