New: (Torznab) Add `magneturl' attribute #1800
Open
+6
−3
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mynameisbogdan
force-pushed
the
torznab-magneturl
branch
3 times, most recently
from
c950098
to
d397e73
Compare
mynameisbogdan
changed the title
magneturl to search responsePrefer Torrent File option
mynameisbogdan
commented
Jul 28, 2023
src/NzbDrone.Core/Indexers/Definitions/Torznab/TorznabSettings.cs
Outdated
Show resolved
Hide resolved
mynameisbogdan
force-pushed
the
torznab-magneturl
branch
from
d397e73
to
02a5482
Compare
mynameisbogdan
changed the title
Prefer Torrent File option
mynameisbogdan
force-pushed
the
torznab-magneturl
branch
from
02a5482
to
c8f7292
Compare
mynameisbogdan
force-pushed
the
torznab-magneturl
branch
4 times, most recently
from
80ef20d
to
c3ffb4c
Compare
mynameisbogdan
force-pushed
the
torznab-magneturl
branch
from
c3ffb4c
to
9ebb927
Compare
|
The way this is done now means no magnet grab will ever get recorded to history in Prowlarr right? Also what happens for results that only have Magnet? Link won't be populated |
Well if the arrs will prefer the magneturl attribute, then yes, no grab history for those. The link will be always populated, I just moved the logic from Results to Controller to I can have MagnetUrl unproxied. Overall, that's why I postponed this PR. It's either these changes or |
mynameisbogdan
changed the title
mynameisbogdan
force-pushed
the
torznab-magneturl
branch
from
9ebb927
to
1d345b9
Compare
|
Kudos, SonarCloud Quality Gate passed! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Database Migration
NO
Issues Fixed or Closed by this PR