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 Timestamp to Output #211

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

Conversation

PatchRequest
Copy link

Added a Timestamp log to the Output to make it easier to read

Example:

[*] [MDNS] 01-Aug-2022 (21:52:38) Poisoned answer sent to xxxx::xxxx:xxxx:xxxx:xxxx for name test.�local

This is a reference to #15

PatchRequest added 3 commits August 1, 2022 21:35
Added a timestamp to console output and cleaner code for output
fix when multi import datetime
@lgandx
Copy link
Owner

lgandx commented Aug 6, 2022

Hello,
This is currently printed in the log files (/logs/Responder-Session.log).
What would be the purpose of printing this in the console output?
Thanks!

@PatchRequest
Copy link
Author

i ran pentest and responder was part of it. Laptop was next to me and i wanted to know if it is still running or maybe got stuck (laptop freezes sometimes). If there would be a timestamp i would have spotted easily when the last broadcast was poisoned.

@PatchRequest
Copy link
Author

its not a big feature but i think more a quality of life so you dont have to look into the logs to see the time

@c0d3x27
Copy link

c0d3x27 commented Sep 23, 2022

its not a big feature but i think more a quality of life so you dont have to look into the logs to see the time

try fixing the errors and it may get merged. This sounds like a good idea

@PatchRequest
Copy link
Author

its not a big feature but i think more a quality of life so you dont have to look into the logs to see the time

try fixing the errors and it may get merged. This sounds like a good idea

fixed

@NeffIsBack
Copy link

Hi, I ran into a similar problem, where I got spammed with requests from one ip and therefore couldn't track if there where new requests, or if Responder just hang up. Is there a possibility you could merge the PR?

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.

4 participants