Use infohash instead of torrent ID in API URLs

[mickvandijke]

[josecelano]

Hi @WarmBeer should we also change the endpoint to download torrent files /torrent/download/{id}?

[josecelano]

In fact, I assume you want to change all the routes that use the id, right? @WarmBeer

pub fn init_routes(cfg: &mut web::ServiceConfig) {
    cfg.service(
        web::scope("/torrent")
            .service(web::resource("/upload").route(web::post().to(upload_torrent)))
            .service(web::resource("/download/{id}").route(web::get().to(download_torrent)))
            .service(
                web::resource("/{id}")
                    .route(web::get().to(get_torrent))
                    .route(web::put().to(update_torrent))
                    .route(web::delete().to(delete_torrent)),
            ),
    );
    cfg.service(web::scope("/torrents").service(web::resource("").route(web::get().to(get_torrents))));
}

[josecelano]

Hi @WarmBeer, let me know what your initial idea was, so I can continue with this issue.

[mickvandijke]

Hi @WarmBeer, let me know what your initial idea was, so I can continue with this issue.

Hi @josecelano ,

Sorry for the late reply. Yes I think we can change all the endpoints from ‘torrent_id’ to ‘info_hash’.