ericpaulbishop / gargoyle

Gargoyle Router Management Utility
http://www.gargoyle-router.com
465 stars 222 forks source link

[devel][plugin-gargoyle-diagnostics] Bug... #964

Closed obsy closed 1 year ago

obsy commented 1 year ago

Please... data should be validated.

image

lantis1008 commented 1 year ago

I'll put in a fix. User already has full control of the router in this case anyway, but I agree I should do better there.

lantis1008 commented 1 year ago

Can you please check this solves the problem? https://github.com/ericpaulbishop/gargoyle/commit/0b904cc87df5234375b0ac77a9fc61f5639179bf

obsy commented 1 year ago

Yes, looks good.