-id optional flag on notify command

[Sim4n6]

Hi ,

In line

sub.Monitor/monitor.py

Line 99 in 8d077ba

command = f'echo "Subdomain {subdomain} has been discovered!" | {notify_binary} -silent -config {notify_api} -id {domain}'

, you are using notify flag -id

According to the help:

-id string[] id to send the notification to (optional)

This one breaks the notify message sending, since id = domain.

I fix it by entirely removing -id {domain} ;-)

[Sim4n6]

The PR #3

[e1abrador]

Hello,

That notify command is intended to be used with Discord configuration as explained on the README.md file (for the way i manage my recon on Discord):

• https://github.com/e1abrador/sub.Monitor#work-plan

If you need to send the notifications on a single channel (even if it's on Discord/Telegram/etc...) I recommend using this line:

    command = f'echo "Subdomain {subdomain} has been discovered for {domain}!" | {notify_binary} -silent -config {notify_api}'

• no -id flag use #3

As I won't add the PR on the original project, let's keep this Issue open (as well as the PR) so people with the same problem can check/manage this on their private use by easily reading this.

Best regards and thanks for opening the issue, have a nice day!

[Sim4n6]

No problem, have a nice day

[Sim4n6]

I have closed the PR since it was made using the main branch of the fork. .. .