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.

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

Adds endpoints for viewing some logs. #2539

Merged
merged 1 commit into from
Feb 10, 2023
Merged

Adds endpoints for viewing some logs. #2539

merged 1 commit into from
Feb 10, 2023

Conversation

afred
Copy link
Contributor

@afred afred commented Feb 1, 2023

  • Adds route /logs for seeing a list of links to available logs.
  • Adds route /logs/[:log_file_name] for viewing plain text rendering of an available log file.
  • To restrict access to some logs, add the filename (or a glob pattern) to LogsController::IGNORE. Attempting to access an ignored log file will respond with a 403 Forbidden.

@afred afred force-pushed the log_routes branch 2 times, most recently from 3001c29 to c90114d Compare February 2, 2023 15:27
@afred afred requested a review from mrharpo February 3, 2023 16:21
@afred afred force-pushed the log_routes branch 3 times, most recently from 31a3e2f to 5564371 Compare February 10, 2023 19:31
* Adds route /logs for seeing a list of links to available logs.
* Adds route /logs/[:log_file_name] for viewing plain text rendering of an available log file.
* To restrict access to some logs, add the filename (or a glob pattern) to  LogsController::IGNORE. Attempting
  to access an ignored log file will respond with a 403 Forbidden.

Also:
* Comments out problematic spec in spec/scripts/cleaner_spec.rb with sporadically fails
  for unrelated reasons, likely due to randomness when running the specs, so the spec
  just needs to be refactored in a major way. But we don't want to keep blocking PRs
  because of it.
@afred afred merged commit a1ac288 into master Feb 10, 2023
@afred afred deleted the log_routes branch February 10, 2023 20:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant