Skip to content

Commit 1200990

Browse files
committed
Fix spec
1 parent 85f4d92 commit 1200990

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

lib/travis/api/app/endpoint/logs.rb

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,13 @@ class Logs < Endpoint
1414
repo = Travis::API::V3::Models::Repository.find(job.repository.id)
1515
repo_can_write = current_user ? !!repo.users.where(id: current_user.id, permissions: { push: true }).first : false
1616

17-
raise LogExpired if !repo.user_settings.job_log_time_based_limit && job.started_at && job.started_at < Time.now - repo.user_settings.job_log_access_older_than_days.days
18-
raise LogAccessDenied if repo.user_settings.job_log_access_based_limit && !repo_can_write
17+
if !repo.user_settings.job_log_time_based_limit && job.started_at && job.started_at < Time.now - repo.user_settings.job_log_access_older_than_days.days
18+
halt 403, { error: { message: "We're sorry, but this data is not available anymore. Please check the repository settings in Travis CI." } }
19+
end
20+
21+
if repo.user_settings.job_log_access_based_limit && !repo_can_write
22+
halt 403, { error: { message: "We're sorry, but this data is not available. Please check the repository settings in Travis CI." } }
23+
end
1924

2025
if !resource || ((job.try(:private?) || !allow_public?) && !has_permission?(job))
2126
halt 404

0 commit comments

Comments
 (0)