refactor: change announce and announceList

[alxhotel]

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

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

• Refactor of the announce and announceList parameters

What changes did you make? (Give an overview)
This PR:

• Allows to have different values in announce and announceList. The current code in master, depends on announceList being false to be able to use the opts.announce; which is good for parsing the torrent. But since we are creating it, I think we should allow to set both values (with no dependency on being false).

For example:

If we use announce = "wss://example1.com" and announceList = [["wss://example2.com"]]

The current code in master, would ignore the value in announce. This PR adds the announce value to announceList.

• Adds more validations to the tracker lists

Which issue (if any) does this pull request address?
Not directly, but the issue seen in #118 would have had a better output. In that particular case (with the new changes), an invalid announceList will not throw an error and just discard the invalid trackers.

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

[hicom150]

I don't know if this change will follow the specification BEP12

If the client is compatible with the multitracker specification, and if the "announce-list" key is present, the client will ignore the "announce" key and only use the URLs in "announce-list".

What do you think?

[alxhotel]

@hicom150 yeah, that's what I was referring to.

The specification tells you what to do for the implementation of reading a torrent but not for the creation.

In fact, the specification implicitly mentions that torrents can have both announce-list and announce set at the same time. The older clients will only care about announce and the newer ones (that implement the multitracker specification) will only care about the announce-list.

So from what I understand, this PR takes both of these client implementations into account.

[github-actions]

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?