[Dnsmasq-discuss] [PATCH] Add new extensible D-Bus signal

Pali Rohár pali.rohar at gmail.com
Thu Jan 9 11:10:22 GMT 2020


On Thursday 09 January 2020 11:15:09 Florent Fourcot wrote:
> Hello Simon,
> 
> On 06/12/2019 10:53, Victorien Molle wrote:
> > For our usage, we need to have more informations sent over D-Bus such as the interface
> > name, the vendor class identifier and the lease expiration time.
> > 
> > To achieve this, we add a new D-Bus signal "DhcpLeaseNotification" which exports the
> > requested informations as a dictionnary.
> > It also has the advantage to be flexible if someone wants to add a new entry in the
> > future.
> > 
> 
> Could you have a look on this patch? It's a new notification feature, with
> more DHCP details inside. We are available to improve it if needed.

Hello! I looked at this API, but is has same problems as old one. It is
not possible to extend it in future (if it would be needed). Could you
please design API in way that can be extended in future? (See my
previous email with details how to achieve it)

-- 
Pali Rohár
pali.rohar at gmail.com



More information about the Dnsmasq-discuss mailing list