Skip to content

feat: add rdp default port check #11525

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

Merged
merged 3 commits into from
Mar 6, 2025
Merged

Conversation

asteria121
Copy link
Contributor

Template / PR Information

  • Created RDP default port checker
  • References:

Template Validation

I've validated this template locally?

  • YES
  • NO

Additional Details (leave it blank if not applicable)

I use my desktop outside via Wake on LAN and RDP. So I use rdp in non-server Windows OS. But using rdp on default port could lead to port scan by attackers. So I made a script which checks rdp is running on default port TCP 3389. If rdp is running and port is not 3389, it is not detected.

Additional References:

@GeorginaReeder
Copy link

Thanks for your contribution @asteria121 ! :)

@pussycat0x pussycat0x self-assigned this Feb 26, 2025
@pussycat0x pussycat0x added the Status: In Progress This issue is being worked on, and has someone assigned. label Feb 26, 2025
@pussycat0x pussycat0x added the good first issue Good for newcomers label Feb 26, 2025
@pussycat0x pussycat0x requested a review from DhiyaneshGeek March 4, 2025 16:24
@pussycat0x pussycat0x added Done Ready to merge and removed Status: In Progress This issue is being worked on, and has someone assigned. labels Mar 4, 2025
@DhiyaneshGeek
Copy link
Member

Hello @asteria121 , Thank you so much for sharing this template with the community and contributing to this project 🍻

You can grab some cool PD stickers over here http://nux.gg/stickers 😄

Join our discord server. It's a great place to connect with fellow contributors and stay updated with the latest developments. Thank you once again

@DhiyaneshGeek DhiyaneshGeek merged commit fbdfea7 into projectdiscovery:main Mar 6, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Done Ready to merge good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants