Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Output seems to be MAC address instead of IP #105

Closed
Elevennails opened this issue May 13, 2022 · 3 comments · Fixed by #107
Closed

Output seems to be MAC address instead of IP #105

Elevennails opened this issue May 13, 2022 · 3 comments · Fixed by #107
Assignees
@vdjagilev
Labels
priority/high High priority issue tech/go Golang type/bug There is a problem with application, something does not work as intended
Milestone
v1.0.2

Comments

@Elevennails
Copy link

Was outputting IP address using Docker container. However, I've installed using Go on a different device and now the outputs only seem to list MAC address.
@Elevennails
Copy link
Author

Docker version was 1.0.0 seems to be an issue with 1.0.1

@vdjagilev vdjagilev added this to v3 May 13, 2022
@vdjagilev vdjagilev added the prop/triage Issues to be triaged label May 13, 2022
@vdjagilev vdjagilev self-assigned this May 13, 2022
@vdjagilev vdjagilev added type/bug There is a problem with application, something does not work as intended priority/high High priority issue tech/go Golang prop/bc-break Property: Breaking Change and removed prop/triage Issues to be triaged labels May 13, 2022
@vdjagilev
Copy link
Owner

Hi,

Thanks for using an app :) I've figured out why is this happening. I've managed to reproduce it by using -O option in nmap (fingerprint OS). An example of such XML output:

<address addr="192.168.1.1" addrtype="ipv4"/>
<address addr="FF:FF:FF:FF:FF:FF" addrtype="mac" vendor="[Snippet]"/>

In this case nmap-formatter takes only the last address. Will fix it.

@vdjagilev vdjagilev moved this to Todo in v3 May 13, 2022
@vdjagilev vdjagilev mentioned this issue May 13, 2022
Closed
vdjagilev added a commit that referenced this issue May 15, 2022
@vdjagilev
#105: Force using ipv4/ipv6 in HostAddress
7b546be
@vdjagilev vdjagilev linked a pull request May 15, 2022 that will close this issue
#105: Force using ipv4/ipv6 in HostAddress #107
Merged
@vdjagilev vdjagilev removed the prop/bc-break Property: Breaking Change label May 15, 2022
vdjagilev added a commit that referenced this issue May 15, 2022
@vdjagilev
#105: Force using ipv4/ipv6 in HostAddress (#107)
a647ad5 
* #105: Force using ipv4/ipv6 in HostAddress

* Codestyle fixes, code simplification
Repository owner moved this from Todo to Released / Merged in v3 May 15, 2022
@vdjagilev
Copy link
Owner

Fixed in v1.0.2 bugfix release
@Elevennails

@vdjagilev vdjagilev added this to the v1.0.2 milestone May 22, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@vdjagilev vdjagilev

Labels
priority/high High priority issue tech/go Golang type/bug There is a problem with application, something does not work as intended
Projects
v3
Status: Released / Merged
Milestone
v1.0.2
Development

Successfully merging a pull request may close this issue.

#105: Force using ipv4/ipv6 in HostAddress vdjagilev/nmap-formatter
2 participants
@vdjagilev @Elevennails