Open jneilliii opened 9 months ago
This looks great, thanks. Can I trust you on your bright blue eyes that this works? I don't have much time to test this at the moment. I would say that your testing is better than mine.
I can tell you I've been using it with one minor issue. Not sure if it's related to newer versions of OctoPrint, or the latest rc I'm testing but if Use these credentials to log in to OctoPrint.
is enabled after logging in remotely it killed the running service. However, I think that was happening prior to my changes because I attempted to access this test device a week or two ago from work and it did the same with basic authentication.
Just confirmed, downgrading the plugin to the latest released version and enabling that option does kill OctoPrint., so unrelated to this change.
I have been testing these changes this morning and would 👍🏻 this for getting merged into this plugin for release.
@jneilliii thanks for adding this! The continual re-auth for basic auth was driving me crazy.