Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Mapping Logic: Exclude Network and Broadcast IPs from Host IP Mapping #20

Open
fatihk25 opened this issue Jul 4, 2023 · 0 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@fatihk25
Copy link

fatihk25 commented Jul 4, 2023

Currently, the mapping logic includes network and broadcast IPs, which is causing issues. This update aims to fix the mapping functionality by excluding network and broadcast IPs, ensuring that only host IPs are used for mapping purposes.

Problem Description:
The current mapping logic for IP addresses includes network and broadcast IPs, which is causing inaccuracies in the mapping results. This issue is illustrated in the example image provided
usage-example

Expected Behavior:
The mapping logic should only consider host IPs for mapping purposes. Network and broadcast IPs should be excluded from the mapping process.

Impact:
The total number of hosts calculated using the updated mapping logic should be 254, as demonstrated in the example image. Out of these hosts, 2 should be marked as online, while 252 should be marked as offline.

Note: This update will improve the accuracy and reliability of the mapping functionality, ensuring that only valid host IPs are considered for mapping purposes

@fatihk25 fatihk25 added the enhancement New feature or request label Jul 4, 2023
@fadhilyori fadhilyori added the question Further information is requested label Jul 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants