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

FTL #408

Merged
merged 35 commits into from
Mar 3, 2017
Merged

FTL #408

merged 35 commits into from
Mar 3, 2017

Conversation

DL6ER
Copy link
Member

@DL6ER DL6ER commented Feb 19, 2017

By submitting this pull request, I confirm the following (please check boxes, eg [X] - no spaces) Failure to fill the template will close your PR:

Please submit all pull requests against the development branch. Failure to do so will delay or deny your request

  • I have read and understood the contributors guide.
  • I have checked that another pull request for this purpose does not exist.
  • I have considered, and confirmed that this submission will be valuable to others.
  • I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  • I give this submission freely, and claim no ownership to its content.

How familiar are you with the codebase?: 10


FTL integration

  • Dynamically fall back to PHP API functions to ensure API does always respond even if FTL is not running for some reason (note: not implemented for Query Log page, but only dashboard)
  • Use PHP API from current devel branch
  • Increase flexibility in getQueryTypes API call
  • Hide top ads list if nothing to display
  • Proper display of activated privacy mode
  • Remove spinner (would otherwise not be removed for zero results)
  • Update tables every 10 seconds
  • Update query types and forward destinations plots every 10 seconds
  • More verbose output in top lists (total number next to percentage in tooltip)
  • Implemented "recentBlocked" keyword for API
  • Ensure compatibility with PHP5 version < 5.4
  • Adjust output format of overTimeData10mins to comply with PHP API
  • Further speedup of Query Log page by showing only the recent 10 minutes by default
  • Hide temperature if FTL is not running and show FTL status
  • Add information to Settings page
  • Adjust error message when loading of query log failed
  • Move processing of domainname, clientIP, clientname and time interval filters for the Query Log page to FTL for speed enhancement
  • Generate link if pi.hole comes up in the Top Domains list
  • Remove resolve DNS names option - this is now enabled by default (daemon will only do it once per day instead of PHP-API which did it on every reloading of the web interface)
  • Add socket timeout of 10 seconds + modification to Settings page since FTL backend supports API_EXCLUDE_CLIENTS filtering with both IP addresses and host names (also mixed)
  • Improved Query Log page

Note that api_PHP.php is a simply copy of the old api.php and unchanged.

This template was created based on the work of udemy-dl.

- Dynamically fall back to PHP API functions to ensure API does always repond even if FTL is not runnung for some reason
- Update PHP API from current devel branch
- Increase flexibility in getQueryTypes API call
- Hide top ads list if nothing to display
- Proper display of activated privacy mode
- Remove spinner (would otherwise not be removed for zero results)
- Update tables every 10 seconds
- Update query types and forward destinations plots every 10 seconds
- More verbose output in top lists (total number next to percentage in tooltip)
- Implemented "recentBlocked" keyword for API
- Ensure compatibility with PHP5 version < 5.4
- Adjust output format of overTimeData10mins to comply with PHP API
- Further speedup of Query Log page by showing only the recent 10 minutes by default
- Hide temperature if FTL is not running and show FTL status
- Add information to Settings page
- Adjust error message when loading of query log failed
- Move processing of domainname, clientIP, clientname and time interval filters for the Query Log page to FTL for speed enhancement
- Generate link if pi.hole comes up in the Top Domains list
- Remove resolve DNS names option - this is now enabled by default (daemon will only do it once per day instead of PHP-API which did it on every reloading of the web interface)
- Add socket timeout of 10 seconds + modification to Settings page since FTL backend supports API_EXCLUDE_CLIENTS filtering with both IP addresses and host names (also mixed)
- Improved Query Log page
	api.php
	scripts/pi-hole/php/FTL.php
	scripts/pi-hole/php/auth.php
	scripts/pi-hole/php/password.php
@DL6ER DL6ER added this to the v3.0 milestone Feb 19, 2017
@DL6ER DL6ER changed the title New/ftl integration [WIP] FTL Feb 19, 2017
Conflicts:
	scripts/pi-hole/php/auth.php
	scripts/pi-hole/php/password.php
@DL6ER DL6ER mentioned this pull request Feb 19, 2017
@@ -1,151 +1,82 @@
<?php
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Github fails to detect that there is basically no change in the PHP API. The old file api.php is not rewritten, but moved to api_PHP.php. This api.php is an entirely new file.

@DL6ER DL6ER changed the title [WIP] FTL FTL Mar 3, 2017
@PromoFaux
Copy link
Member

PromoFaux commented Mar 3, 2017

Approved

Approved with PullApprove

@DL6ER DL6ER merged commit fff8427 into devel Mar 3, 2017
@DL6ER DL6ER deleted the new/FTL-integration branch March 3, 2017 12:41
@pralor-bot
Copy link

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/pi-hole-3-0-is-here-featuring-our-new-ftl-engine/2828/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants