kylon / Sharedown

Electron application to download Sharepoint videos (especially meant for students)
GNU General Public License v3.0
349 stars 40 forks source link

Login page refresh periodically #39

Closed massimostanzione closed 2 years ago

massimostanzione commented 2 years ago

Login page, with Microsoft (University) login, automatically refreshes each 7 seconds, whether mail/password are inserted or not. The only way to avoid this is to type and enter credentials in less than 7 seconds. After three refreshes, Pupeeter error/Unable to find a valid donor response!

Version 4.2.2, on Linux Mint 20.2

Settings:

massimostanzione commented 2 years ago

Update: changing the value of Sharedown login module (and setting mail/password fields):

kylon commented 2 years ago

a regression from 0f645bef6a5fe79cb9f635d907281dc49f1ae064

i ll work on it and attach a test version here

kylon commented 2 years ago

please test Sharedownt1.zip

unpack open a terminal in unpacked folder type npm i or yarn install to download required files type npm start or yarn start to start sharedown

massimostanzione commented 2 years ago

Done, tried all the settings combinations mentioned above and the timeout problem seems fixed now. (just for the record, still present the issue with Sharedown login module: University [MS 365]).

Thank you!

kylon commented 2 years ago

login modules are not universal, if you have to type the password means your login page has different input, and according to what you say the simple login module is working so that is the one compatible with your login page

about the response error, you get that only with ms 365?

massimostanzione commented 2 years ago

login modules are not universal, if you have to type the password means your login page has different input, and according to what you say the simple login module is working so that is the one compatible with your login page

Oh yes, now that I think about it I was convinced that my login had something to do with MS365, nevermind :)

about the response error, you get that only with ms 365?

Yes, the other methods work without any issue.

kylon commented 2 years ago

ok, i think it is just the wrong module causing the error.

i ll push a new release later today with the fix above, thanks for testing.