ericpaulbishop / gargoyle

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

Disable Layer7 Protocol UI elements interaction #740

Closed d3fz closed 6 years ago

d3fz commented 6 years ago

Even though it's been extensively advised not to use Layer7 Protocol for now (due to a guaranteed router bootloop), every now and then someone pops up in the forum reporting/asking for help (mostly unaware new users).

This simple CSS rule could prevent users from accidentally shooting themselves in the foot (bricking their routers), at least for now. Since it's all done via CSS, it shouldn't break any functionality and can be enabled/disabled in a heartbeat, whenever necessary. Works for both Desktop/Mobile versions.

/* Disable Controls / Block events  */
#rule_app_protocol_label,
#rule_app_protocol_type,
#exception_app_protocol_label,
#exception_app_protocol_type,
#use_app_protocol:enabled,
#app_protocol_label,
#app_protocol
{
    pointer-events: none;
    touch-action: none;
    user-select: none;
    opacity: 0.5;
    border-color: #ccc;
    box-shadow: none;
}

QoS Download/Upload

d19d48e7-8136-4c1c-a04f-387ba4567816

Restrictions

e366c22d-2317-4224-afc0-902851cb8e7f

This could also be applied to the "Attempt to Preserve Settings" checkbox option, but since I've seen some people taking the risk and using this option even on big updates (when highly advised not to), I'll just leave as it is. fb7cefe0-5679-4ebb-97c0-7021c52b0ac5

ispyisail commented 6 years ago

re:"Attempt to Preserve Settings"

I wonder if you can compare versions to enable/disable?

or have this feature disabled on official builds but enabled on test builds / self builds?

lantis1008 commented 6 years ago

See extensive discussion https://github.com/ericpaulbishop/gargoyle/issues/446

ericpaulbishop commented 6 years ago

I'm going to merge this... I think it's finally time to ditch layer7. It just doesn't work anymore, and I'm not going to bother porting the l7 iptables module to the latest kernel version for the new release.

lantis1008 commented 6 years ago

Once everything is up and running I’ll look at nDPI (again).