NeverWise / plugin.video.dplay

Kodi unofficial plugin for Dplay
GNU General Public License v3.0
28 stars 17 forks source link

Compatibilità Python 3 + retrocompatibilità Python 2 #22

Closed cttynul closed 4 years ago

cttynul commented 5 years ago

Come da titolo e come accennato nella pull request https://github.com/NeverWise/script.module.neverwise/pull/1

Colgo l'occasione di questa pull più che per mergare il codice in sé quanto per proporre una nuova feature (non disponibile nella mia branch in oggetto), ovvero l'eventualità di aggiungere i canali live editi Discovery fruibili sul sito web e sull'app ufficiale, ie:

channel

Tuttavia i link di streaming di risposta al JSON sono con scadenza in timestamp (circa 24h dalla generazione dello stesso): https://sbshdlu5-lh.akamaihd.net/i/sbshdl_4@810998/master.m3u8?hdnts=st=1569426271~exp=1569512671~acl=/*~hmac=9975c0a120e5b518583d5db7d4a8db7a1f95de1b8f2252ffce5412fff6b4ccf3&mux_audio=true

Aggirabile rimuovendo gli argomenti della chiamata dell'm3u8: https://sbshdlu5-lh.akamaihd.net/i/sbshdl_4@810998/master.m3u8

La domanda è: Mantenere o meno l'eventuale timestamp di scadenza e, soprattutto, come gestire i canali Plus, esclusivi dei clienti paganti (Discovery Channel, Discovery Science ndr.) un semplice messaggio in cui si avvisa l'utente che il canale non sia fruibile?

NeverWise commented 5 years ago

Per i canali live la cosa migliore sarebbe quella di emulare il comportamento del sito web o dell'app ufficiale e cioè mettere il timestamp se loro lo mettono, in questo modo se decidono di cambiare qualcosa nell'url, l'addon è sempre aggiornato. Mentre per i canali Plus l'ideale sarebbe gestire username e password, ma mi rendo conto che non è un lavoro così immediato, per cui direi che una notifica è sicuramente la soluzione migliore anche perché ho notato che ci possono essere video a pagamento e video gratuiti all'interno dello stesso programma televisivo e anche in questo caso andrebbe notificato.

P.S. Ho aggiunto dei commenti al tuo codice se è OK faccio il merge.

phate89 commented 4 years ago

Non so se stai ancora aggiornando questa pr ma un metodo migliore al posto di usare try catch è usare https://kodi.wiki/view/Add-on:Kodi_Six Ha dei wrapper per le librerie kodi che gestiscono automaticamente la conversione e se preferisci non usarli ha funzioni specifiche per il decoding/encoding

NeverWise commented 4 years ago

Non so se stai ancora aggiornando questa pr ma un metodo migliore al posto di usare try catch è usare https://kodi.wiki/view/Add-on:Kodi_Six Ha dei wrapper per le librerie kodi che gestiscono automaticamente la conversione e se preferisci non usarli ha funzioni specifiche per il decoding/encoding

Io non sto aggiornando questo pull request. Comunque la tua mi sembra un'ottima idea, sentiti libero di contribuire se ti va ;-)