Skip to content

handle multiple output case#261

Merged
Mzack9999 merged 2 commits intodevfrom
handle_multiple_output_case
Feb 16, 2024
Merged

handle multiple output case#261
Mzack9999 merged 2 commits intodevfrom
handle_multiple_output_case

Conversation

@dogancanbakir
Copy link
Copy Markdown
Member

@dogancanbakir dogancanbakir commented Feb 1, 2024

Closes #260

before:

$ go run . -d example.com -list example-subdomains.txt -r resolvers.txt -t 35 -mcmd '--processes 2'

       __        ________        __       
  ___ / /  __ __/ _/ _/ /__  ___/ /__ ___
 (_-</ _ \/ // / _/ _/ / -_)/ _  / _ \(_-<
/___/_//_/\_,_/_//_//_/\__/ \_,_/_//_/___/

                projectdiscovery.io

[INF] Current shuffledns version v1.0.9 (latest)
[INF] Creating temporary massdns output file: /var/folders/hx/f8qcbgj91795xdrdnb55qkdc0000gn/T/shuffledns-1057597848/cmtnemfa2ua2oc3ud9ag
[INF] Executing massdns on example.com
[INF] Massdns execution took 2.006099084s
[INF] Started parsing massdns output
[ERR] Could not run massdns: could not parse massdns output: could not open massdns output file: open /var/folders/hx/f8qcbgj91795xdrdnb55qkdc0000gn/T/shuffledns-1057597848/cmtnemfa2ua2oc3ud9ag: no such file or directory
[INF] Finished resolving. Hack the Planet!

after:

$ go run . -d example.com -list example-subdomains.txt -r resolvers.txt -t 35 -mcmd '--processes 2'

       __        ________        __       
  ___ / /  __ __/ _/ _/ /__  ___/ /__ ___
 (_-</ _ \/ // / _/ _/ / -_)/ _  / _ \(_-<
/___/_//_/\_,_/_//_//_/\__/ \_,_/_//_/___/

                projectdiscovery.io

[INF] Current shuffledns version v1.0.9 (latest)
[INF] Creating temporary massdns output file: /var/folders/hx/f8qcbgj91795xdrdnb55qkdc0000gn/T/shuffledns-3866732922/cmtne3na2ua2l2fs27ng
[INF] Executing massdns on example.com
[INF] Massdns execution took 1.878241333s
[INF] Started parsing massdns output
[INF] Massdns output parsing completed
[INF] Started removing wildcards records
[INF] Wildcard removal completed
[INF] Finished enumeration, started writing output
localhost
[INF] Finished resolving. Hack the Planet!

@dogancanbakir dogancanbakir self-assigned this Feb 1, 2024
@dogancanbakir dogancanbakir linked an issue Feb 7, 2024 that may be closed by this pull request
could not parse massdns output when --processes 2 specified #260
Closed
Mzack9999
Mzack9999 approved these changes Feb 16, 2024
View reviewed changes
Copy link
Copy Markdown
Member

@Mzack9999 Mzack9999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - --processes forks the massdns process leading to multiple output files

@Mzack9999 Mzack9999 merged commit 5688527 into dev Feb 16, 2024
@Mzack9999 Mzack9999 deleted the handle_multiple_output_case branch February 16, 2024 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Mzack9999 Mzack9999 Mzack9999 approved these changes

@ehsandeep ehsandeep Awaiting requested review from ehsandeep

Assignees

@dogancanbakir dogancanbakir

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

could not parse massdns output when --processes 2 specified

2 participants

@dogancanbakir @Mzack9999