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.

Sign up for GitHub

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

Jump to bottom

JSON output field include host information in the ip field for SSL Templates #2966

Closed
ehsandeep opened this issue Dec 5, 2022 · 0 comments · Fixed by projectdiscovery/tlsx#132, #3002 or #3006
Closed

JSON output field include host information in the ip field for SSL Templates #2966

ehsandeep opened this issue Dec 5, 2022 · 0 comments · Fixed by projectdiscovery/tlsx#132, #3002 or #3006
Assignees
@tarunKoyalwar
Labels
Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.

Comments

@ehsandeep
Copy link
Member

Nuclei version:

master | dev

Current Behavior:

host is being reported instead of ip

Expected Behavior:

write ip information in the ip field.

Steps To Reproduce:

nuclei -id tls-version -u hackerone.com -json | jq -r .ip

                     __     _
   ____  __  _______/ /__  (_)
  / __ \/ / / / ___/ / _ \/ /
 / / / / /_/ / /__/ /  __/ /
/_/ /_/\__,_/\___/_/\___/_/   v2.7.9

		projectdiscovery.io

[INF] Using Nuclei Engine v2.7.9 (latest)
[INF] Using Nuclei Templates 9.3.0 (latest)
[INF] Templates added in last update: 70
[INF] Templates loaded for scan: 1
[INF] Targets loaded for scan: 1
hackerone.com
@ehsandeep ehsandeep added the Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors. label Dec 5, 2022
@tarunKoyalwar tarunKoyalwar self-assigned this Dec 5, 2022
This was referenced Dec 5, 2022
Closed
Merged
@tarunKoyalwar tarunKoyalwar added Status: Review Needed The issue has a PR attached to it which needs to be reviewed Status: Blocked There is some issue that needs to be resolved first. and removed Status: Review Needed The issue has a PR attached to it which needs to be reviewed labels Dec 5, 2022
@tarunKoyalwar tarunKoyalwar linked a pull request Dec 5, 2022 that will close this issue
fix resolved ip and other minor improvements projectdiscovery/tlsx#132
Merged
@Mzack9999 Mzack9999 removed the Status: Blocked There is some issue that needs to be resolved first. label Dec 8, 2022
@ehsandeep ehsandeep linked a pull request Dec 8, 2022 that will close this issue
tlsx go mod update #3002
Merged
4 tasks
@ehsandeep ehsandeep added the Status: Completed Nothing further to be done with this issue. Awaiting to be closed. label Dec 8, 2022
@ehsandeep ehsandeep linked a pull request Dec 8, 2022 that will close this issue
Bugfix release #3006
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tarunKoyalwar tarunKoyalwar

Labels
Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Bug Inconsistencies or issues which will cause an issue or problem for users or implementors.
Projects
None yet
Milestone
No milestone
3 participants
@ehsandeep @Mzack9999 @tarunKoyalwar