Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login using Selenium (very rough but works) #2

Closed
wants to merge 3 commits into from
Closed

Login using Selenium (very rough but works) #2

wants to merge 3 commits into from

Conversation

aaronjamt
Copy link

When running this script, it will now open a browser window using Selenium and allow the user to log in to the actual Floatplane website. It's very rough code (I haven't done any Go programming before, at least that I remember), minimally commented, and doesn't save your session (you have to log in and enter your 2FA code, if you have one, every single time), but it works.

@aaronjamt
Copy link
Author

Update: I found a Go module to save credentials to a user's keyring so now it will do so whenever possible (and try to retrieve from the keyring first before prompting the user to log in)

@Flonja
Copy link
Owner

Flonja commented Mar 27, 2023

I like the keyring solution that you contributed, but relying on Selenium is something I'm not willing to put in a library. Selenium seems to have extra (native?) dependencies that just added extra bloat. Proper (frontend) authentication should probably be handled outside of this project.

@aaronjamt
Copy link
Author

How about a copy+paste Javascript command you could enter into the developer console to grab the SID?

@Flonja
Copy link
Owner

Flonja commented Mar 27, 2023

I have put your pr in another repository. Please look there how I would have implemented your idea

@aaronjamt
Copy link
Author

All that needs to be done for this repo then is adding integration with the other one somehow so I'll close this issue since that's a broader topic.

@aaronjamt aaronjamt closed this Mar 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants