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

Newznab - Wrong Parsing logic for id's (OMG) non-standard IDs #1261

Closed
1 task done
Fotguedes opened this issue Dec 26, 2022 · 3 comments
Closed
1 task done

Newznab - Wrong Parsing logic for id's (OMG) non-standard IDs #1261

Fotguedes opened this issue Dec 26, 2022 · 3 comments
Labels
Area: Indexer Issue is related to indexers. Type: Bug Issue is a bug

Comments

@Fotguedes
Copy link

Fotguedes commented Dec 26, 2022

Is there an existing issue for this?

  • I have searched the existing open and closed issues

Current Behavior

I'm having the same problem that sysmoon14 had in #656

This item from OMG via Prowlarr look like this:

<newznab:attr name="category" value="5000" />
<newznab:attr name="category" value="5040" />
<newznab:attr name="rageid" value="0" />
<newznab:attr name="tvdbid" value="0" />
<newznab:attr name="imdb" value="0000000" />
<newznab:attr name="tmdbid" value="0" />
<newznab:attr name="traktid" value="0" />

but the same item direct from OMG API returns:

<newznab:attr name="category" value="5000"/>
<newznab:attr name="category" value="5040"/>
<newznab:attr name="tvmaze" value="55350"/>
<newznab:attr name="tvdb" value="402910"/>

I guess because OMG don't use the attr name as "tvdbid" it is no able to parse.

Expected Behavior

Prowlarr be able to parse the expected ID's

Steps To Reproduce

No response

Environment

- OS: Linux
- Prowlarr: 1.0.0.2171
- Docker Install: No
- Using Reverse Proxy: No

What branch are you running?

Develop

Trace Logs?

None

AB#4556

@Fotguedes Fotguedes added Status: Needs Triage New Issue needing triage Type: Bug Issue is a bug labels Dec 26, 2022
@bakerboy448 bakerboy448 changed the title IMDB, TVDB. OMDB IDs Not Exposed in API Results Newznab - Wrong Parsing logic for id's Dec 26, 2022
@bakerboy448
Copy link
Contributor

Spec is https://newznab.readthedocs.io/en/latest/misc/api/#list-of-attributes

  • imdb
  • rageid

code is

releaseInfo.TvdbId = GetIntAttribute(item, "tvdbid");

need to treat tvdb as tvdbid...probably should do similar for the various other names.

also interesting to see what other sites look like

@bakerboy448 bakerboy448 added the Area: Indexer Issue is related to indexers. label Dec 26, 2022
@bakerboy448
Copy link
Contributor

bakerboy448 commented Dec 27, 2022

Starrs

Sonarr Newznab IDs

  • tvdbid
  • rageid

Radarr Newznab IDs

  • imdb

Indexers

OMG

Default attributes:
* language   -> Language of the NZB (Defaults to english if unknown) 
* category   -> Category of the NZB - show twice for general and specific category
* size   -> Size of the NZB in Bytes
* coverurl   -> Link to coverurl (if it exists)

Extended attributes:
* group   -> Group the NZB was posted to
* imdb   -> IMDB id of the NZB (if it exists)
* tvmaze   -> TVMaze id of the NZB (if it exists)
* rageid   -> TVRage id of the NZB (if it exists)
* tvdb   -> TVDB id of the NZB (if it exists)
* info   -> Link to nfo file (if it exists)
* usenetdate   -> Time the NZB was posted to usenet
* guid   -> ID of the NZB on the site
* grabs   -> Number of downloads so far
* haspretime   -> Shows if the NZB has a pretime
* nuked   -> Shows if the NZB is nuked (0 or 1)
* nukereason   -> Shows the nuked reason (only if nuked = 1)
* password   -> Shows the password of the release (if the release has a password)

short name

Extended Attributes
Using the attrs tag and a comma separated list of supported values, extended information can be returned in the search results. 
For example ?attrs=files,poster,group. Note that not every attribute is available for every release type. Below is a list of some of the supported attributes. To return all known attributes per release use the parameter ?extended=1. See the API specification for a full list.

files
poster
group
team
grabs
password
comments
usenetdate
info
year
season
episode
rageid
tvmazeid
tvdbid
tvtitle
tvairdate
video
audio
resolution
framerate
language
subs
imdb
imdbscore
imdbtitle
imdbtagline
imdbplot
imdbyear
imdbdirector
imdbactors
genre
artist
album
publisher
tracks
coverurl
backdropcoverurl
review

@Qstick
Copy link
Contributor

Qstick commented Dec 27, 2022

Yea, I think OMG is just off from standard on their attributes... I'll add some parsing for them, but it's clearly wrong comparing to others.

@bakerboy448 bakerboy448 removed the Status: Needs Triage New Issue needing triage label Dec 27, 2022
@bakerboy448 bakerboy448 changed the title Newznab - Wrong Parsing logic for id's Newznab - Wrong Parsing logic for id's (OMG) non-standard IDs Dec 27, 2022
@Qstick Qstick closed this as completed in 587a73f Dec 27, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Area: Indexer Issue is related to indexers. Type: Bug Issue is a bug
Projects
None yet
Development

No branches or pull requests

3 participants