Merge pull request #97 from anacrolix/option-announce

Make the announce info field optional
This commit is contained in:
Igor Katson 2024-03-24 09:03:11 +00:00 committed by GitHub
commit 24feb60001
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 5 additions and 7 deletions

View file

@ -200,7 +200,7 @@ pub async fn create_torrent<'a>(
let info_hash = compute_info_hash(&info).context("error computing info hash")?;
Ok(CreateTorrentResult {
meta: TorrentMetaV1Owned {
announce: b""[..].into(),
announce: None,
announce_list: Vec::new(),
info,
comment: None,

View file

@ -721,10 +721,7 @@ impl Session {
.map(|t| ByteString(t.into_bytes()))
.collect();
let info = TorrentMetaV1Owned {
announce: trackers
.first()
.cloned()
.unwrap_or_else(|| ByteString(b"http://retracker.local/announce".to_vec())),
announce: trackers.first().cloned(),
announce_list: vec![trackers],
info: storrent.info,
comment: None,

View file

@ -29,7 +29,8 @@ pub fn torrent_from_bytes<'de, ByteBuf: Deserialize<'de>>(
/// A parsed .torrent file.
#[derive(Serialize, Deserialize, Debug, Clone)]
pub struct TorrentMetaV1<BufType> {
pub announce: BufType,
#[serde(skip_serializing_if = "Option::is_none")]
pub announce: Option<BufType>,
#[serde(
rename = "announce-list",
default = "Vec::new",
@ -59,7 +60,7 @@ impl<BufType> TorrentMetaV1<BufType> {
if self.announce_list.iter().flatten().next().is_some() {
return itertools::Either::Left(self.announce_list.iter().flatten());
}
itertools::Either::Right(once(&self.announce))
itertools::Either::Right(self.announce.iter())
}
}