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

added support for TXT,CNAME,MX,AAAA and A records #1072

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bogdan-dt
Copy link

No description provided.

@mandreko-ts
Copy link

mandreko-ts commented Aug 4, 2024

For what it's worth, I set this up tonight for an SE engagement that I'm working on. I used Mailgun for SMTP, and was able to create all of the DNS records for domain verification easily. When I attempted with SendGrid, it didn't ever seem to verify, despite the DNS resolving properly. I'm wondering if SendGrid doesn't like the periods at the end of the hostnames (which are part of the DNS spec)

Regardless, this worked great with Mailgun. I had TXT, MX, and CNAME records. The only caveat that I found was that with my domain with the glue records, I had to manually add the A records to match the glue records, which would normally appear in any normal DNS server automatically.

So for a phishing domain of example.com, you'd have in your dns_records.yaml:

a_records:
  ns1.example.com.:
    - ttl: 360
      value: "1.2.3.4"
  ns2.example.com.:
    - ttl: 360
      value: "1.2.3.4"

EDIT:

As I was testing it, I also found that the automatic DNS resolution stopped working for my proxied hosts when moving to this build. I did have to specify A records for each proxied host. However, Evilginx2 makes it easy with the phishlets get-hosts <phishlet> command. It listed each of my proxied hostnames, and I added an A record for each one. Afterwards, everything worked again.

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.

2 participants