Ignore None values in kwargs

[ffourcot]

Hello,

We have a regression on code based on previous pyroute2 version. It looks like parameters given in kwargs are coded, even for None values.

I used strict_mode / ext_ack for a better error message from the kernel, but they are not mandatory to reproduce:

# this one is working
In [15]: IPRoute(ext_ack=True, strict_check=True).rule('add', table=10, priority=32000)
Out[15]: 
[{'attrs': [('NLMSGERR_ATTR_UNUSED', None)],
  'header': {'length': 36,
   'type': 2,
   'flags': 256,
   'sequence_number': 255,
   'pid': 3199518433,
   'error': None,
   'target': 'localhost',
   'stats': Stats(qsize=0, delta=0, delay=0)},
  'error': 0,
  'event': 'NLMSG_ERROR'}]

# adding src=None breaks it
In [16]: IPRoute(ext_ack=True, strict_check=True).rule('add', table=10, priority=32001, src=None)
---------------------------------------------------------------------------
NetlinkError                              Traceback (most recent call last)
Cell In [16], line 1
----> 1 IPRoute(ext_ack=True, strict_check=True).rule('add', table=10, priority=32001, src=None)

File pyroute2/pyroute2/iproute/linux.py:2493, in IPRoute.__getattr__.<locals>.synchronize_generic(*argv, **kwarg)
   2491 else:
   2492     task = collect_op
-> 2493 return self.event_loop.run_until_complete(task())

File /usr/lib/python3.11/asyncio/base_events.py:653, in BaseEventLoop.run_until_complete(self, future)
    650 if not future.done():
    651     raise RuntimeError('Event loop stopped before Future completed.')
--> 653 return future.result()

File pyroute2/pyroute2/iproute/linux.py:2483, in IPRoute.__getattr__.<locals>.synchronize_generic.<locals>.collect_op()
   2482 async def collect_op():
-> 2483     return await symbol(*argv, **kwarg)

File pyroute2/pyroute2/iproute/linux.py:2337, in RTNL_API.rule(self, command, **kwarg)
   2335 if command == 'dump':
   2336     return request.response()
-> 2337 return [x async for x in request.response()]

File pyroute2/pyroute2/iproute/linux.py:2337, in <listcomp>(.0)
   2335 if command == 'dump':
   2336     return request.response()
-> 2337 return [x async for x in request.response()]

File pyroute2/pyroute2/netlink/nlsocket.py:566, in NetlinkRequest.response(self)
    565 async def response(self):
--> 566     async for msg in self.sock.get(
    567         msg_seq=self.msg_seq,
    568         terminate=self.terminate,
    569         callback=self.callback,
    570     ):
    571         if self.dump_filter is not None and not self.match_one_message(
    572             self.dump_filter, msg
    573         ):
    574             continue

File pyroute2/netlink/core.py:468, in AsyncCoreSocket.get(self, msg_seq, terminate, callback, noraise)
    466         enough = True
    467 if not noraise and error:
--> 468     raise error

NetlinkError: (22, 'Invalid source address')

We have a lot of code assuming that None values are ignored in NLA encoding, should we update our code to use recent pyroute2 version or can you ignore this kind of values again?

Thanks

[svinota]

I will fix that to keep the API compatible, sure thing.

Thanks for notifying.