Skip to content

Conversation

@krazak
Copy link
Contributor

@krazak krazak commented Apr 1, 2024

What is the purpose of this pull request? (put an "X" next to item)

[ ] Documentation update
[ ] Bug fix
[x ] New feature
[ ] Other, please explain:

What changes did you make? (Give an overview) Added timestamps to the event logs

Which issue (if any) does this pull request address? It helps to correlate events that occur with your bittorrent client's logs.

Is there anything you'd like reviewers to focus on? No.

New tracker log:

2024-04-01T21:02:04.570Z bittorrent-tracker:server new server {"http":true,"stats":true,"trustProxy":false,"udp":true,"ws":true}
2024-04-01T21:02:04.670Z bittorrent-tracker:server listen (port: 8000 hostname: { http: undefined, udp4: undefined, udp6: undefined })
2024-04-01T21:02:04.889Z bittorrent-tracker:server listening
2024-04-01T21:02:04.890Z HTTP tracker: http://localhost:8000/announce
2024-04-01T21:02:04.904Z UDP tracker: udp://0.0.0.0:8000
2024-04-01T21:02:04.904Z UDP6 tracker: udp://localhost:8000
2024-04-01T21:02:04.904Z WebSocket tracker: ws://localhost:8000/
2024-04-01T21:02:04.904Z Tracker stats: http://localhost:8000/stats
2024-04-01T21:02:12.162Z start: 172.29.0.7:6881
2024-04-01T21:04:39.877Z start: 172.29.0.13:51414

@welcome
Copy link

welcome bot commented Apr 1, 2024

🙌 Thanks for opening this pull request! You're awesome.

@himabindugit
Copy link

Any update on reviewing the PR?

Copy link
Member

@SilentBot1 SilentBot1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable to me.

@SilentBot1 SilentBot1 merged commit f6a7993 into webtorrent:master Apr 11, 2024
@welcome
Copy link

welcome bot commented Apr 11, 2024

🎉 Congrats on getting your first pull request landed!

@webtorrent-bot
Copy link

🎉 This PR is included in version 11.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants