cisagov / cyhy-core

Core code for Cyber Hygiene (CyHy)
Other
7 stars 9 forks source link

Correctly reference `IPSet` #71

Closed dav3r closed 2 years ago

dav3r commented 2 years ago

๐Ÿ—ฃ Description

This PR fixes a bug by correctly referencing IPSet() instead of netaddr.IPSet().

๐Ÿ’ญ Motivation and context

When the netaddr import was modified in https://github.com/cisagov/cyhy-core/commit/a17267a2ccaf56f4e3ad51ccffbd2e8674e023f7, we overlooked changing this reference to IPSet. Without this change, you will get: NameError: global name 'netaddr' is not defined

๐Ÿงช Testing

I verified that this code change allowed me to run cyhy-import and have it run correctly and without the NameError: global name 'netaddr' is not defined that I was getting previously.

โœ… Pre-approval checklist