Refactoring Ip.py to return proper boolean values

[pUrGe12]

Proposed change

Currently the ip.py file has functions that are return boolean values to determine valid IP addresses. In functions like is_ipv4_range, its using and with netaddr.IPNetwork() which returns an IPNetwork object and not a boolean. This hasn't affected any functionality because its always being used as if is_ipv4_range(ip) which results in true as long as the function doesn't return False, but for tests it will be necessary.

Type of change

• New core framework functionality
• Bugfix (non-breaking change which fixes an issue)
• Code refactoring without any functionality changes
• New or existing module/payload change
• Localization improvement
• Dependency upgrade
• Documentation improvement

Checklist

• I've followed the contributing guidelines
• I've run make pre-commit, it didn't generate any changes
• I've run make test, all tests passed locally

[securestep9]

please don't mix the tests with core functionality refactoring in one PR.

[pUrGe12]

I'll keep this one only to refactoring the ip.py file to make it suitable for the tests.