Webserver graphs are not displaying correctly

arevindh / pihole-speedtest

Pihole Speedtest Mod

https://arevindh.github.io/pihole-speedtest/

MIT License

218 stars 22 forks source link

Webserver graphs are not displaying correctly #86

Closed yanniedog closed 2 years ago

yanniedog commented 3 years ago

First: I really love your mod. Fantastic. I'll buy you a coffee after this post.

After installing this mod, the Speedtest graph always loads correctly, but it has caused a problem with how the original graphs on the pihole dashboard load

These graphs only show up if I move my mouse cursor over these them, or the graphs automatically refresh. They don't display automatically when the page loads (even after hitting 'refresh')

Please see the attached screenshots

arevindh commented 3 years ago

Which browser and version are you using ? Any console errors ?

yanniedog commented 3 years ago

Google Chrome Version 91.0.4472.101 (Official Build) (64-bit). I have rechecked the webserver using the Pi's own browser (Chromium), and the same thing happens

The Pi appears to be working normally. What do you mean by console errors? Is there a log file I can send you?

AftDawn commented 3 years ago

Hi there, my speedtest graph is not even showing up

i'm also getting CSP errors inside of Brave

AftDawn commented 3 years ago

I tried removing img-src from the CSP but that made every image not load (i don't know anything about php or the CSP part of html) so i added it back again.

arevindh commented 3 years ago

@AftDawn seems like your test does not yet run. Can you help me with the result of <pihole-ip>/api.php?getSpeedData24hrs&PHP Login to pihole before accessing the url (Please remove/hide IP address before posting here )

AftDawn commented 3 years ago

nevermind, it just seems that i needed to wait a day. tho i'm still having to mouse over the charts to get the info

amorshc commented 2 years ago

Adding to this. I would like to see labels on the sides for Ping vs Speed. As you can clearly see my graph looks a bit weird because normally it's pretty steady, but when a huge spike in ping occurs, it throws the graph all funky.

arevindh commented 2 years ago

@amorshc added a new update to enable labels .

amorshc commented 2 years ago

@amorshc added a new update to enable labels .

Looks great! I love the functionality of this, and appreciate you adding the labels!

Also adding that after the update the speed test did show (using the built in version of Chromium on my raspberry pi) no graphs as stated above in this issue. After letting everything settle in for 5-10 minutes I refreshed the page and I see all graphs fine - I believe this actually to be a browser issue not an issue with the script.