Traffic Monitoring - Save frequency - issue

[v.y.k]

Asuswrt-Merlin Version 3004.388.6 - Asus RT-AX86U Pro

Traffic Monitoring.
Save frequency choice "Every 1 week" does not get persistent.
After using "Apply" button, save frequency reverts back to the previous saved choice.

Similar issue was found in Asuswrt-Merlin Version 386.12_4 - Asus RT-AC68U

 

[dave14305]

Asuswrt-Merlin Version 3004.388.6 - Asus RT-AX86U Pro

Traffic Monitoring.
Save frequency choice "Every 1 week" does not get persistent.
After using "Apply" button, save frequency reverts back to the previous saved choice.

Similar issue was found in Asuswrt-Merlin Version 386.12_4 - Asus RT-AC68U

I see this behavior. The nvram variable is only 2 characters, so it cannot store the 168 (hours) required for the Every 1 week option. Nothing obvious has changed, so Asus may be doing stricter nvram validation. Every 3 days (72 hours) sticks, as expected.

 

[v.y.k]

So maximum value is 99 hours ~ 4.125 days, using direct nvram setting.
Should "Every 1 week" choice be removed from drop down?

 

[bbunge]

Why would you want to go so long? A reboot/power failure your stats from the 99 hours, three days or other l o n g setting are poof... gone. Stick with one hour. Does not write that much data to storage....

 

[Tech9]

poof... gone.

In stock Asuswrt, but @v.y.k is using Asuswrt-Merlin and saves the data to file.

Short period of 1h is better. On reboot it may lose up to 1h of data only. Sometimes people make non-logical choices. I don't know why.

 

[v.y.k]

Why would you want to go so long? A reboot/power failure your stats from the 99 hours, three days or other l o n g setting are poof... gone. Stick with one hour. Does not write that much data to storage....

Should "Every 1 week" choice be removed from drop down, since it does not work?

 

[Tech9]

I think it has to be removed. It's a firmware bug - impossible setting.

 

[dave14305]

Looks like it goes back to the early days of 382.x

shared: adjust allowed length for various nvram parameter that were s… · RMerl/asuswrt-merlin.ng@83b3727

…et to CKN_STR_DEFAULT (meaning 0, preventing from changing these through the webui)

github.com