luci-base: return ipv6 lease time from getExpiry()

This commit introduces a streamlined method for retrieving lease time
 information on IPv6 interfaces.

While IPv4 interfaces typically have a single lease time (available
 in the interface data), IPv6 interfaces commonly feature multiple
 lease times due to factors like delegated prefixes or multiple /64 subnet.

Users typically receive only one prefix delegation from their ISP, leading
 this PR to the retrieval of lease time from the first available prefix
 or IPv6 address.

To enhance usability and provide users with a more comprehensive view
 of lease time information, however, alternative approaches
 should be discussed.

Suggestions include returning an array of lease times or aggregating
 the various possible values using methods like min or max.

These enhancements aim to improve the clarity the lease time
 management on IPv6 interfaces.

Signed-off-by: hitech95 <nicveronese@gmail.com>
This commit is contained in:
hitech95 2024-04-27 11:25:42 +02:00 committed by Paul Donald
parent e01f393924
commit 2bb1e06b52
1 changed files with 20 additions and 5 deletions

View File

@ -2141,12 +2141,27 @@ Protocol = baseclass.extend(/** @lends LuCI.network.Protocol.prototype */ {
*/
getExpiry: function() {
var u = this._ubus('uptime'),
d = this._ubus('data');
d = this._ubus('data'),
v6_prefixes = this._ubus('ipv6-prefix'),
v6_addresses = this._ubus('ipv6-address');
if (typeof(u) == 'number' && d != null &&
typeof(d) == 'object' && typeof(d.leasetime) == 'number') {
var r = d.leasetime - (u % d.leasetime);
return (r > 0 ? r : 0);
if (typeof(u) == 'number' && d != null) {
// DHCPv4 or leasetime in data
if(typeof(d) == 'object' && typeof(d.leasetime) == 'number') {
var r = d.leasetime - (u % d.leasetime);
return (r > 0 ? r : 0);
}
// DHCPv6, we can have multiple IPs and prefixes
if (Array.isArray(v6_prefixes) || Array.isArray(v6_addresses)) {
var prefixes = [...v6_prefixes, ...v6_addresses];
if(prefixes.length && typeof(prefixes[0].valid) == 'number') {
var r = prefixes[0].valid;
return (r > 0 ? r : 0);
}
}
}
return -1;