google / capirca

Multi-platform ACL generation system
Apache License 2.0
778 stars 209 forks source link

Remove ipaddress unnecessary dependency #309

Closed mirceaulinic closed 2 years ago

mirceaulinic commented 2 years ago

It seems like Capirca targets Python 3.6+. The ipaddress library is available natively beginning with Python 3.3+, or having it listed under the project requirements pulls the PyPI library which hasn't seen a release in almost 3 years and has a number of security issues:

rdsharma commented 2 years ago

Looks like this breaks the Windows build unfortunately. Any idea why?

mirceaulinic commented 2 years ago

@rdsharma seems like 1571b13 helped.

mirceaulinic commented 2 years ago

@rdsharma Would you know when a new version of Capirca will be released on PyPI, to include this fix? Thanks!