Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restrict Repack upgrades to the same release group #946

Closed
markus101 opened this issue Nov 16, 2015 · 5 comments
Closed

Restrict Repack upgrades to the same release group #946

markus101 opened this issue Nov 16, 2015 · 5 comments

Comments

@markus101
Copy link
Member

If a repack comes out Sonarr should only upgrade to it if the original file is from the same release group, a proper should apply to any release group, but what can happen with internal releases is a scene release comes out (no issues) and later a repack for an internal release comes out and Sonarr happily upgrades to it.

Proper: Fix for another release groups mistake
Repack: Fix for same release groups mistake

@aptalca
Copy link

aptalca commented Jan 27, 2016

Just had this issue with x-files

Already had the regular/official scene release, a day later a repack for an internal came out, and sonarr replaced the original with the internal repack.

Along the same lines, perhaps sonarr should pay attention to whether proper releases match the original in terms of whether they are internal or not.

For instance, let's say there is a regular scene release and an internal. Sonarr downloads the internal. Later, a proper may come out for the original release by a different group. In the current state, Sonarr would replace the existing internal (which is probably just fine) with the proper. Perhaps Sonarr should not replace the internal with a non-internal proper.

@scootied
Copy link

scootied commented Feb 9, 2016

Just had the same problem with the X-Files again (ref aptalca above). Code for the same release name + proper should fix the issue.

@NMe84
Copy link

NMe84 commented Feb 9, 2016

A proper is always something that another group releases to fix the first group's release. Repacks are fixes a group makes for their own releases. Same-group logic would need to only apply to repacks, not propers. The whole point of propers is that they are released by another group.

The upgrade for X-files you just got (FLEET -> 2HD proper) was valid by all counts.

@Taloth
Copy link
Member

Taloth commented Feb 9, 2016

@markus101 Do we really want to spend effort on this? Especially since p2p won't give a rats ass about repack/proper, nor does scene care about p2p.
It downloaded another release due to a conflict with an iNTERNAL, so what? If that's such a big issue, disable upgrading to propers.
Otherwise you're gonna have a discussion about what's better, official release or iNTERNAL.

@markus101
Copy link
Member Author

I think its worth it for a repack, same as we do for anime (only get v2 if v1 was from the same release group), for a proper, I don't think we change anything.

Of course the complication comes with proper vs repack, since we don't differentiate in code between the two.

ta264 pushed a commit to ta264/Sonarr that referenced this issue Dec 5, 2019
* Update inCinemas column to abide by the short date set in settings (Sonarr#511)

* Set default port to 587 for Email Settings, should help with all the people with gmail

* set SSL to true by default
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants