openscopeproject / TrguiNG

Remote GUI for Transmission torrent daemon
GNU Affero General Public License v3.0
278 stars 33 forks source link

Feature request: tracker list editing, dedupe when adding trackers to avoid rejection #134

Closed Willian-Zhang closed 5 months ago

Willian-Zhang commented 6 months ago

expected: adding new trackers actual: "tracker edit failed" with no detailed reason given

qu1ck commented 6 months ago

What is your transmission server version?

Willian-Zhang commented 6 months ago

What is your transmission server version?

4.0.5

qu1ck commented 6 months ago

Can you post screenshot of the error? Also what do you put in trackers list? It's possible the data is malformed and server is rejecting it, not TrguiNG.

Willian-Zhang commented 6 months ago

I tried removing empty lines from new added trackers, but same error, an observation is that there are empty lines from an existing torrent (before editing).

Willian-Zhang commented 6 months ago

And I just found out cmd+v for pasting on macOS doesn't work in track list editing

Willian-Zhang commented 6 months ago

I tried removing empty lines from new added trackers, but same error, an observation is that there are empty lines from an existing torrent (before editing).

I'm aware of the line saying empty line between tiers. Just not sure if that's causing some parsing problem

Willian-Zhang commented 6 months ago

Also what do you put in trackers list?

a fully qualified tracker, copied from existing one.

e.g. before edit

AAA
BBB

CCC
DDD

after

AAA
BBB

CCC
DDD
BBB
qu1ck commented 6 months ago

Transmission does not allow repeating tracker.

Willian-Zhang commented 6 months ago

It's possible the data is malformed and server is rejecting it, not TrguiNG.

It would not appear on transmission remote.app

Willian-Zhang commented 6 months ago

Transmission does not allow repeating tracker.

Alright! Sorry for my quick judge, must have been the other app had done some dedupeing

qu1ck commented 5 months ago

Fixed in v1.2.0