andrebrait / service.subtitles.legendastv

Other
9 stars 6 forks source link

Mensagem de erro de autenticação e novo gerador de link de download. #4

Closed RobertoHigor closed 3 years ago

RobertoHigor commented 3 years ago

Corri uma mensagem de erro sem explicação que, ao olhar os logs, me devolvia o seguinte erro:

2021-03-07 21:46:18.750 T:2752   ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
    - NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
    Error Type: <type 'exceptions.TypeError'>
    Error Contents: argument 1 must be string or buffer, not None
    Traceback (most recent call last):
    File "C:\Users\Higor\AppData\Roaming\Kodi\addons\service.subtitles.legendastv\service.py", line 357, in <module>
    params['lang'])
    File "C:\Users\Higor\AppData\Roaming\Kodi\addons\service.subtitles.legendastv\service.py", line 134, in Download
    f.write(FileContent)
    TypeError: argument 1 must be string or buffer, not None
    -->End of Python script error report<--

A primeira coisa que eu fiz foi corrigir o código que pega o Download ID, pois estava caindo no if e retornando 0. Criei um regex para pegar o código direto da url, não necessitando realizar um get na página inteira para verificar uma seção específica (que creio causar erros ao baixar pack)

Depois da alteração, pude verificar que o erro na verdade se tratava de autenticação. O request de login retorna uma mensagem sem acentos, no entanto o código checava pela frase "Usuário ou senha inválida", não caindo no else.

Aproveitei para adicionar uma mensagem básica de erro caso alguém tente baixar uma legenda sem estar autenticado. image

andrebrait commented 3 years ago

Cara, não recebi a notificação desse PR. Me desculpe. Vou revisar em breve