Skip to content
This repository has been archived by the owner on Mar 2, 2020. It is now read-only.

Use the domain_audit table for top domain audit filter #215

Merged
merged 3 commits into from
Oct 15, 2019

Conversation

AzureMarker
Copy link
Contributor

The domain audit data has been moved to the gravity database: pi-hole/pi-hole#2832

Extra features are added to the repository for future endpoints to add domains to the audit list.

Signed-off-by: Mcat12 <newtoncat12@yahoo.com>
The audit log file is no longer used and has been removed.

Signed-off-by: Mcat12 <newtoncat12@yahoo.com>
@AzureMarker AzureMarker requested a review from a team August 6, 2019 22:13
@codecov
Copy link

codecov bot commented Aug 10, 2019

Codecov Report

Merging #215 into development will increase coverage by 0.14%.
The diff coverage is 74.6%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #215      +/-   ##
===============================================
+ Coverage        69.24%   69.38%   +0.14%     
===============================================
  Files               91       92       +1     
  Lines             6262     6360      +98     
  Branches          1155     1163       +8     
===============================================
+ Hits              4336     4413      +77     
- Misses             983      998      +15     
- Partials           943      949       +6
Impacted Files Coverage Δ
src/env/config/file_locations.rs 58.1% <ø> (+0.41%) ⬆️
src/env/file.rs 87.5% <ø> (-0.74%) ⬇️
src/ftl/memory_model/domain.rs 54.16% <25%> (ø) ⬆️
src/services/domain_audit/repository.rs 69.23% <69.23%> (ø)
src/routes/stats/top_domains.rs 68.94% <83.33%> (+1.53%) ⬆️
src/routes/stats/database/top_domains_db.rs 82.37% <84.61%> (+1.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c91ceb6...f4fee01. Read the comment docs.

@AzureMarker AzureMarker merged commit 84105b4 into development Oct 15, 2019
@AzureMarker AzureMarker deleted the feature/audit-log-db branch October 15, 2019 02:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants