New Spy and Spy bot – very very long loading time

Hi,

Usually there is a php timeout (15 seconds) that many providers implement to prevent script that take too much time in execution to blocks the site. It seems that you don’t have it as you need to move the plugin for re-activate it.

New Spy scans all the IP (so lot of data), but spy-bot only the last week of them, so for not much data (if you have normal visits).

What about spy? Did it runs correctly?

you can test for errors by adding this

<?php
error_reporting(E_ALL);
ini_set(‘display_errors’, 1);
/*

instead of this

<?php
/*

in newstatpress.php

Errors will be inserted in you site at video, so test it quickly (and then rollback to original php file) as the users that enter the site can find some errors at video too.

Hi,

I’ve just tested 0.6.8 on 3 sites using responsive and non-responsive themes and I encounterd no problems with Spy, New Spy or Spy Bot.

The performance timings generated by p3-profiler show that the plugin is very fast.

Great update, Davide

Good Morning ice00,

I have a test installation on the same server. Because I didn’t want to play around with my live blog i did the following:
– added the code above as described
– got error messages on all pages. messages were different on all pages
– adding
– I deactivated all plugins -> no error messages
– added WP_DEBUG & WP_DEBUG_LOG true to wp_config

being in the admin area I get:

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Declaration of Graphene_Description_Walker::start_el() should be compatible with Walker::start_el(&$output, $object, $depth, $args, $current_object_id = 0) in /www/htdocs/XXXXXXX/YYYYYYY/wp-content/themes/graphene/includes/theme-menu.php on line 49

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

on the blog i get:

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

Strict Standards: Declaration of Graphene_Description_Walker::start_el() should be compatible with Walker::start_el(&$output, $object, $depth, $args, $current_object_id = 0) in /www/htdocs/XXXXXXX/YYYYYYY/wp-content/themes/graphene/includes/theme-menu.php on line 49

Strict Standards: Only variables should be passed by reference in /www/htdocs/XXXXXXX/YYYYYYY/wp-includes/pomo/mo.php on line 210

the debug.log shows the following when clicking on Overview in admin area:
Undefined offset: 1 in /www/htdocs/XXXXXXX/YYYYYYY/wp-content/plugins/newstatpress/newstatpress.php on line 735

and in details:

Undefined variable: chartData in /www/htdocs/XXXXXXX/YYYYYYY/wp-content/plugins/newstatpress/newstatpress.php on line 1561

Undefined variable: maxValue in /www/htdocs/XXXXXXX/YYYYYYY/wp-content/plugins/newstatpress/newstatpress.php on line 1561

new spy runs for about 90 secs and I get the same messages as “on the blog”

I hope this may be useful to you in any way ??

Best regards
Seth

and not being logged in I get this message on the blog
Notice: Trying to get property of non-object in /www/htdocs/XXXXXXX/YYYYYYY/wp-content/plugins/newstatpress/newstatpress.php on line 1965

The first two block of errors comes from WP itself.

735: list($null,$q)=explode(“?”,$url);

this is not related to your problem. It is a warning (the function try to extract feeds information and if not present gives that warning (I will try to resolve the warning in next release)

1561: $data=$chartData.”&chxt=y&chxl=0:|0|”.$maxValue;

Not related to your problem, but good catching. It is a peace of old chart code remaining to remove from the source (it does nothing). I will fix this in 0.6.9 for sure.

1965: ‘$userdata->user_login’,

can you confirm the line above at that position?
It is not related to newspy, but it is the code that insert the data to database.

As you can work with a test instalation, could you try this?

Locate:
function iriNewStatPressNewSpy() {

and after this:

// Number of distinct ip (unique visitors)
  $NumIP = $wpdb->get_var("
    SELECT count(distinct ip)
    FROM $table_name
    WHERE spider=''"
  );

add:
$NumIP =1;

This limit the number of IP to show (maybe you should get only one page).
Let me know if with this newspy it is working (that means it is using little data)

Newspy is working/uses less data. Takes about ~ 25sec to load now.

This function is showing me 157 pages to select (in pagination).
Perhaps you could add an option to limit these pages to last week / last month / last 50 entries / …

BTW since the last update 0.6.9 I get in function spy not all flags shown but the follwoing message

xxxlitespeed web server
litespeed technologies is not responsible for administration and contents of this web site! .png

Ok, so it is because you have lot of data and PHP timing blocks the execution to prevent server low performance.
I will see how to implement such features for reducing the timing interval.

If the country is not determined from WP, it is used the service:
https://api.hostip.info/country.php?ip=

that gives the country.
It is possible that that external sevice has some problems and gives that message instead of a valid country

Sorry, I was busy the last days.
I appreciate your efforts and I am looking forward to the next releases.

Perhaps you could add an option to reduce the spider data in the database but to keep the “real” visitors to keep the totalvisits count.

If I prune old data the total visits get reduced as well. This is why I dont use this feature.

Yes, I can add such features ??

SetheDeBlade: Perhaps you could add an option to reduce the spider data in the database but to keep the “real” visitors to keep the totalvisits count.

ice00: Yes, I can add such features ??

Me: Or better still, delete all the old visitor and/or spider data on request, and just keep the counts for each. Less old data to lug around, historic counts maintained.

Great(!!!) plugin, by the way. Thank you!