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

Some improvements to the code #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

snooze6
Copy link

@snooze6 snooze6 commented Apr 11, 2024

Hey there,

I had to use dnslivery on a engagement and there were some changes that I had to implement, I leave them here if you want to merge them.

  • Added resolution to type A and AAAA dns queries because some dns servers require that to forward the request. I found that when using nslookup file-txt.save.domain.com 8.8.8.8 dnslivery received a type A query for save.domain.com and if that was unanswered the dns server 8.8.8.8 returned timeout. Resolving everything to localhost worked perfectly.
  • Added case insensitive domain resolution. In one scenario a client had a dns server that randomized upercase and lowercase letters, probably to avoid dns c&cs. I just added a quick patch to make the code case insensitive.
  • Added CLM mode. If Powershell's language mode is set to constrained it is not possible to execute the default save payload because method invocation is supported only on core types in the constrained language mode.
  • Added 53 UDP socket listener to show the port as open.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant