medusa plugin by Marquez96 · Pull Request #2282 · Flexget/Flexget

Marquez96 · 2018-12-21T15:27:33Z

Motivation for changes:

I was updating myinfrastructure with medusa, and i tried to connect with flexget via sickbear plugin, but the api of sickbear of medusa was outdated and i create a new plugin specific for medusa

Detailed changes:

I created the plugin based on old sickbear plugin, i reformat and created all the rest requests of that

Implemented feature requests:

Feathub https://feathub.com/Flexget/Flexget/+65

Config usage if relevant (new plugin or updated schema):


templates:
  medusa:
    configure_series:
      from:
        medusa:
          base_url: http://127.0.0.1
          port: 8081
          username: username
          password: password
          only_monitored: True

# ---------- General config
    configure_series:
      from:
        medusa:
          base_url: string
          port: string default 8081
          username: string
          password: string
          only_monitored: boolean default False
          included_ended: boolean default False

Log and/or tests output (preferably both):

DEBUG MODE 
2018-12-21 23:24 DEBUG    medusa        updating_series processing show: {'title': 'Westworld', 'airs': 'Sunday 0:00AM', 'showType': 'series', 'rating': {'imdb': {'votes': 317083, 'rating': '8.8'}}, 'nextAirDate': None, 'cache': {'banner': '/opt/medusa/cache/images/tmdb/63247.poster.jpg', 'poster': '/opt/medusa/cache/images/tmdb/63247.poster.jpg'}, 'language': 'es', 'genres': ['Drama', 'Mystery', 'Western', 'Ciencia ficción', 'Science-Fiction'], 'id': {'slug': 'tmdb63247', 'tmdb': 63247, 'imdb': 'tt0475784', 'trakt': None}, 'classification': '', 'config': {'dvdOrder': False, 'sports': False, 'airByDate': False, 'qualities': {'allowed': [2, 4, 8, 32, 64, 128, 256, 512, 1024, 2048, 4096, 8192, 16384, 32768], 'preferred': []}, 'paused': False, 'defaultEpisodeStatus': 'Wanted', 'anime': False, 'release': {'requiredWords': [], 'ignoredWords': []}, 'aliases': [], 'subtitlesEnabled': False, 'location': '/mnt/media/series/westworld', 'seasonFolders': False, 'scene': False}, 'network': 'HBO', 'year': {'start': 2016}, 'countries': ['UNITED STATES'], 'type': 'Scripted', 'runtime': 62, 'akas': {}, 'plot': 'Westworld ...', 'country_codes': ['us'], 'status': 'Continuing', 'indexer': 'tmdb'}

INFO MODE - Silence
2018-12-20 20:04 VERBOSE  details       updating_series Produced 16 entries.

To Do:

It's recomendable to detect qualities in medusa, now i only can choose for only_monitories and ended series

liiight · 2018-12-23T08:01:53Z

flexget/plugins/input/medusa.py

+        )
+
+        try:
+            api_key = json.loads(task.requests.post(


you can use native requests formatting here:

Suggested change

api_key = json.loads(task.requests.post(

api_key = task.requests.post('{}/authenticate'.format(base_url), json=body_auth, headers=self.headers).json()

liiight · 2018-12-23T08:05:51Z

flexget/plugins/input/medusa.py

+                authorization='Bearer ' + api_key
+            )
+
+            series = json.loads(task.requests.get(


look above for optimal request syntax. Also, ideally pass url param via its own dict:

Suggested change

series = json.loads(task.requests.get(

params = {

'limit': 1000

}

series = task.requests.get('{}/series'.format(base_url), params=params).json()

liiight · 2018-12-23T08:07:02Z

flexget/plugins/input/medusa.py

+    schema = {
+        'type': 'object',
+        'properties': {
+            'base_url': {'type': 'string'},


you can set expected format to uri:

Suggested change

'base_url': {'type': 'string'},

'base_url': {'type': 'string', 'format': 'uri'},

liiight · 2018-12-23T08:07:58Z

flexget/plugins/input/medusa.py

+
+        entries = []
+        for show in series:
+            log.debug('processing show: {}'.format(show))


let logger do its own string interpolation:

Suggested change

log.debug('processing show: {}'.format(show))

log.debug('processing show: %s', show)

Marquez96 · 2018-12-26T13:17:03Z

The requested changes have been made,

Thank you for those observations

liiight · 2018-12-26T14:40:13Z

flexget/plugins/input/medusa.py

+        'additionalProperties': False
+    }
+
+    headers = {


this is redundant, as it is implicitly added when you use the json keyword when sending requests

liiight · 2018-12-26T14:40:57Z

flexget/plugins/input/medusa.py

+            log.debug('processing show: %s', show)
+            if (show['config']['paused'] and config.get('only_monitored')) or \
+                    show['status'] == 'Ended' and not config.get('include_ended'):
+                continue


maybe add a debug message here?

Marquez96 · 2018-12-26T15:10:39Z

Debug message added and remove the default headers

liiight · 2018-12-26T15:11:44Z

Thanks! Can you please update the wiki?

Marquez96 · 2018-12-26T15:33:11Z

Yes, of course,
https://flexget.com/Plugins/Medusa

Medusa plugin added

90f9160

liiight requested changes Dec 23, 2018

View reviewed changes

fix styles and optimizate code

2adf3d4

liiight requested changes Dec 26, 2018

View reviewed changes

debug message and modify default headers

a41b0c1

liiight approved these changes Dec 26, 2018

View reviewed changes

liiight merged commit 1dab338 into Flexget:develop Dec 26, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

medusa plugin#2282

medusa plugin#2282
liiight merged 3 commits intoFlexget:developfrom
Marquez96:develop

Marquez96 commented Dec 21, 2018 •

edited

Loading

Uh oh!

liiight Dec 23, 2018

Uh oh!

liiight Dec 23, 2018

Uh oh!

liiight Dec 23, 2018

Uh oh!

liiight Dec 23, 2018

Uh oh!

Marquez96 commented Dec 26, 2018

Uh oh!

liiight Dec 26, 2018

Uh oh!

liiight Dec 26, 2018

Uh oh!

Marquez96 commented Dec 26, 2018

Uh oh!

liiight commented Dec 26, 2018

Uh oh!

Marquez96 commented Dec 26, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

	api_key = json.loads(task.requests.post(
	api_key = task.requests.post('{}/authenticate'.format(base_url), json=body_auth, headers=self.headers).json()

-            series = json.loads(task.requests.get(
+			params = {
+				'limit': 1000
+			}
+            series = task.requests.get('{}/series'.format(base_url), params=params).json()

	'base_url': {'type': 'string'},
	'base_url': {'type': 'string', 'format': 'uri'},

	log.debug('processing show: {}'.format(show))
	log.debug('processing show: %s', show)

Conversation

Marquez96 commented Dec 21, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Motivation for changes:

Detailed changes:

Implemented feature requests:

Config usage if relevant (new plugin or updated schema):

Log and/or tests output (preferably both):

To Do:

Uh oh!

liiight Dec 23, 2018

Choose a reason for hiding this comment

Uh oh!

liiight Dec 23, 2018

Choose a reason for hiding this comment

Uh oh!

liiight Dec 23, 2018

Choose a reason for hiding this comment

Uh oh!

liiight Dec 23, 2018

Choose a reason for hiding this comment

Uh oh!

Marquez96 commented Dec 26, 2018

Uh oh!

liiight Dec 26, 2018

Choose a reason for hiding this comment

Uh oh!

liiight Dec 26, 2018

Choose a reason for hiding this comment

Uh oh!

Marquez96 commented Dec 26, 2018

Uh oh!

liiight commented Dec 26, 2018

Uh oh!

Marquez96 commented Dec 26, 2018

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Marquez96 commented Dec 21, 2018 •

edited

Loading