recording user last activity timestamp

lib/travis/api/app/endpoint.rb

@@ -117,6 +117,7 @@ def authorizer
       end
 
       def auth_for_repo(id, type)
+        current_user&.touch
         permission = authorizer.for_repo(id, type)
         halt 403, { error: { message: "We're sorry, but you're not authorized to perform this request" } } unless permission
       rescue Travis::API::V3::AuthorizerError

lib/travis/api/v3/access_control/user.rb

@@ -7,6 +7,7 @@ class AccessControl::User < AccessControl::Generic
     def initialize(user)
       user                = Models::User.find(user.id) if user.is_a? ::User
       @user               = user
+      user.touch
       @access_permissions = user.permissions.where(user_id: user.id)
       super()
     end

lib/travis/api/v3/models/user.rb

@@ -87,6 +87,10 @@ def installation
       @installation = Models::Installation.find_by(owner_type: 'User', owner_id: id, removed_by_id: nil)
     end
 
+    def touch
+      update(last_activity_at: Time.now) if last_activity_at.nil? || Time.now.utc - last_activity_at > 300
+    end
+
     def github?
       vcs_type == 'GithubUser'
     end

lib/travis/model/user.rb

@@ -47,6 +47,10 @@ def with_email(email_address)
     end
   end
 
+  def touch
+    update(last_activity_at: Time.now) if last_activity_at.nil? || Time.now.utc - last_activity_at > 300
+  end
+
   def token
     tokens.first.try(:token)
   end

spec/lib/model/user_spec.rb

@@ -86,6 +86,15 @@ def user(payload)
     end
   end
 
+  describe 'last activity' do
+    it 'contains last activity' do
+      user.save!
+      expect(user.last_activity_at).to be_nil
+      user.touch
+      expect(user.last_activity_at).to_not be_nil
+    end
+  end
+
   describe 'avatar_url' do
     it "returns avatar url if it's present" do
       user.avatar_url = 'foo'

spec/v3/service_spec.rb

@@ -37,15 +37,18 @@ module Routes
   context 'when forcing authentication' do
     before { Travis.config.force_authentication = true }
     after { Travis.config.force_authentication = false }
+    before { User.last.update!(last_activity_at: nil) }
 
     it 'does not allow access without authentication' do
       get '/v3/examples'
       expect(last_response.status).to eq 403
     end
 
     it 'does allow access with authentication' do
+      expect(User.last.last_activity_at).to be_nil
       get '/v3/examples', {}, auth_headers
       expect(last_response.status).to eq 200
+      expect(User.last.last_activity_at).to_not be_nil
     end
 
     it 'does allow access with log token' do