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

Do not log requests to the health endpoint #1897

Merged

Conversation

acelaya
Copy link
Member

@acelaya acelaya commented Oct 20, 2023

Closes #1896 1896

@acelaya acelaya force-pushed the feature/disable-health-endpoint-logs branch from 88f9add to 55885b0 Compare October 20, 2023 07:33
@codecov
Copy link

codecov bot commented Oct 20, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d419b9d) 96.73% compared to head (5bde273) 96.73%.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #1897   +/-   ##
==========================================
  Coverage      96.73%   96.73%           
  Complexity      1306     1306           
==========================================
  Files            244      244           
  Lines           4530     4530           
==========================================
  Hits            4382     4382           
  Misses           148      148           
Files Coverage Δ
module/Core/src/Model/DeviceType.php 100.00% <100.00%> (ø)
module/Rest/src/Action/HealthAction.php 100.00% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@acelaya acelaya merged commit 2a782ab into shlinkio:develop Oct 20, 2023
20 checks passed
@acelaya acelaya deleted the feature/disable-health-endpoint-logs branch October 20, 2023 18:44
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.

Disable logs for health check endpoint
1 participant