gdomenici / plugin.video.pcloud-video-streaming

Plugin for Kodi (Matrix and above) to provide video streaming from pcloud.com
GNU General Public License v3.0
14 stars 3 forks source link

Update pcloudapi.py #30

Closed lukkie112 closed 3 years ago

lukkie112 commented 3 years ago

Fix Login Problem to pCloud Account

gdomenici commented 3 years ago

Hi @lukkie112 . Thanks for your PR. I am afraid I cannot merge it as it is, because

  1. eapi.pcloud.com is not mentioned anywhere in the PCloud official docs as an official API endpoint. If you Google for that, it's mentioned as the "European endpoint" but it doesn't seem to be official. I wouldnt like to use an undocumented endpoint in the released version of the plugin.
  2. But, more importantly, it doesn't seem to work for me. If I try the change you have in your PR, I am unable to authenticate.

What do you suggest?

Kind regards,

Guido

lukkie112 commented 3 years ago

Hi! It is so, that my fix is ONLY for Users with EU Sever Location, as I The eAPI I have read out with developer tools on Firefox an it works fine for me Perhaps you can do a fix for booth Api's?

It is the same when i I have to use WebDav on some Explorer Tools on Android. There i have too use eWebDav instead of Webdav (this information is fom customer service)

gdomenici-verimatrix commented 3 years ago

Hi! It is so, that my fix is ONLY for Users with EU Sever Location, as I The eAPI I have read out with developer tools on Firefox an it works fine for me Perhaps you can do a fix for booth Api's?

It is the same when i I have to use WebDav on some Explorer Tools on Android. There i have too use eWebDav instead of Webdav (this information is fom customer service)

Hi @lukkie112

Thanks for your clarification. In that case, the PR is OK, but we also need to add a setting to the settings screen to let users choose which of the two servers (api or eapi) do they want to use. If you know how to do that, it would be awesome if you also added some code to expose this setting. Would you like to help with that?

Kind regards,

Guido

lukkie112 commented 3 years ago

Hi! I myself can not help to fix this problem. But I asked a Developer Friend for help. He has an idea and is working for the fix

lukkie112 commented 3 years ago

Hi! Now I Close this pull request because there is a new one from me: #31