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

Overhaul stats: Add announced Peer to UDP core announce event #1384

Open
josecelano opened this issue Mar 18, 2025 · 0 comments
Open

Overhaul stats: Add announced Peer to UDP core announce event #1384

josecelano opened this issue Mar 18, 2025 · 0 comments
Assignees
Labels
- Developer - Torrust Improvement Experience Code Cleanup / Refactoring Tidying and Making Neat

Comments

@josecelano
Copy link
Member

Depends on: #1341

IMPORTANT: This has to be implemented after #1341.

I'm refactoring events to include more metadata:

pub enum Event {
    UdpConnect { context: ConnectionContext },
    UdpAnnounce { context: ConnectionContext },
    UdpScrape { context: ConnectionContext },
}

#[derive(Debug, PartialEq, Eq)]
pub struct ConnectionContext {
    client_socket_addr: SocketAddr,
    server_socket_addr: SocketAddr,
}

It would also be convenient to include information about the announce request, in the event, like the peer.

Read this for more information about how to include the announced peer data.

@josecelano josecelano added - Developer - Torrust Improvement Experience Code Cleanup / Refactoring Tidying and Making Neat labels Mar 18, 2025
@josecelano josecelano self-assigned this Mar 18, 2025
@josecelano josecelano changed the title Overhaul stats: add announced Peer to UDP core announce event Overhaul stats: Add announced Peer to UDP core announce event Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
- Developer - Torrust Improvement Experience Code Cleanup / Refactoring Tidying and Making Neat
Projects
None yet
Development

No branches or pull requests

1 participant