search

GliderGeek / pocket2rm

get articles from read-later platform pocket to the remarkable paper tablet
MIT License
184 stars 15 forks source link

setup stops at "Insert consumerKey" #9

Closed tomhull closed 3 years ago

tomhull commented 3 years ago

I have followed the install process but it appears to hang after I enter my consumerKey.

I'm using Ubuntu 20.04 LTS

technimad commented 3 years ago

I can confirm this issue. I'm experiencing the same.

stiliajohny commented 3 years ago

same issue here

stiliajohny commented 3 years ago

however, the first time it might hang after the key is imported try again it should be open a browser window. then you auth and it might get stuck again. close the browser window try again

tomhull commented 3 years ago

thank you, I repeated the processes, and knowing that a browser was needed I installed one (it's a headless machine) I can confirm this is now working for me.

GliderGeek commented 3 years ago

@tomhull : thanks for reporting back. i never thought about the case of a headless machine. (out of curiosity: what was the reason to do this?)

the browser is needed for the OAuth flow of pocket unfortunately.

things i can improve on my side:

UPDATE: currently the error of opening the browser is not handled https://github.com/GliderGeek/pocket2rm/blob/master/cmd/pocket2rm-setup/main.go#L60

tomhull commented 3 years ago

No problem, I'm running windows 10 with the linux subsystem.

On Tue, 1 Dec 2020 at 16:55, GliderGeek notifications@github.com wrote:

@tomhull https://github.com/tomhull : thanks for reporting back. i never thought about the case of a headless machine. (out of curiosity: what was the reason to do this?)

things to improve on my side:

  • list necessity of browser
  • improve logging (log when key is accepted, attempting to open browser, maybe even error out when this does not succeed)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/GliderGeek/pocket2rm/issues/9#issuecomment-736680247, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAWELHP3CZJMRS5ONDE6X3TSSUNXTANCNFSM4UECQCRQ .

GliderGeek commented 3 years ago

added the necessity of a browser in the prerequisites. if anything else is going on, this can be re-opened