search

Derek-K / telegraf-speedtest

Using official SpeedTest.net cli for telegraf
GNU General Public License v3.0
32 stars 10 forks source link

Pull Req #6

Closed risb0r closed 4 years ago

risb0r commented 4 years ago

Hi There,

Hope you are ok with the changes. All good if you are not.

Derek-K commented 4 years ago

@risb0r I cannot accept this pull request for the following reasons.

1) I believe the user should accept OOKLA's license agreement when using their binary, that is the reason I do not want to include " --accept-license --accept-gdpr" which will bypass it completely. That's the reason specifically mentioned in Step 2 - Run 'speedtest' once (to accept the license) 2) Branding of my original work (modification to my Grafana dashboard 12428) and removed the reference from your update