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

Extract package torrust-tracker-client #1078

Closed
josecelano opened this issue Nov 8, 2024 · 2 comments · Fixed by #1081
Closed

Extract package torrust-tracker-client #1078

josecelano opened this issue Nov 8, 2024 · 2 comments · Fixed by #1081
Assignees
Labels
- Developer - Torrust Improvement Experience Code Cleanup / Refactoring Tidying and Making Neat
Milestone

Comments

@josecelano
Copy link
Member

Relates to:

I have recently extracted a package tracker-client. This package contains all related to tracker clients with two functionalities:

  • Low-level tracker clients (UDP and HTTP). This is a library.
  • High-level tracker console clients (UDP, HTTP and Checker). These are console apps.

In our weekly meeting, we discussed it, and we decided to split this package into two packages too:

  • bittorrent-tracker-client
  • torrust-tracker-client

The reason is bittorrent-tracker-client are generic clients, and torrust-tracker-client is a Torrust console app.

packages/tracker-client

  • Crate: bittorrent-tracker-client
  • It contains only a library with low-level clients.

console/tracker-client

  • Crate: torrust-tracker-client
  • It contains the high-level console clients only.

NOTICE: This console clients will be merged in the future in a single one.

cc @da2ce7

@da2ce7
Copy link
Contributor

da2ce7 commented Nov 11, 2024

@josecelano what do you think about using the /tools/ instead of /console/ for the client?

@josecelano
Copy link
Member Author

@josecelano what do you think about using the /tools/ instead of /console/ for the client?

Hi @da2ce7 I would use console until we have more tools, unless you already know about other tools we would like to implement. Since the crate name is different, we can change it later.

Alternatively, we could use tools/console/tracker-client if you think console is too specific to be in the root folder.

@josecelano josecelano self-assigned this Nov 13, 2024
@josecelano josecelano linked a pull request Nov 13, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

2 participants