SukkaW / fast-cidr-tools

cidr-tools but up to 20x faster
http://npm.im/fast-cidr-tools
MIT License
7 stars 0 forks source link

overlap is broken #1

Closed jmicco closed 2 months ago

jmicco commented 2 months ago

overlap is broken! It doesnt matter how much faster it is!

x.overlap(['10.119.0.0/16'], ['10.119.10.6/24']) true

x.overlap(['10.119.0.0/16'], ['172.20.250.229/31', '10.119.10.6/24']) false

EXPECTED_RESULT: The second call to overlap should return true ACTUAL_RESULT: The second call to overlap returns false

SukkaW commented 2 months ago

Fixed in https://github.com/SukkaW/fast-cidr-tools/commit/39c45a0a881dd2c778599f3f81e64add347f0b78 and the bugfix is included in version 0.3.1!