Unable to login to on-prem from electron app/headless version
@n-riesco @chriddyp Stuck in a deadlock situation for On Prem login from Falcon.
To Clarify: This issue is not valid for the version we ship with On Prem. This is because we control the environment variables on On Prem.
Explanation:
- Open the electron app or web app. Try to login to an on-prem installation.
- It tries to check the username against
PLOTLY_API_URLand fails becausePLOTLY_API_URLis not set initially. https://github.com/plotly/falcon-sql-client/blob/0d774bc43600dcd219bf4f378c4a6c517d00f70e/backend/routes.js#L272 - Since it fails, the user is not logged in and hence
PLOTLY_API_URLis not set because this request requires user to log-in to be able to save tosettings.yaml.
Proposed solution:
-
For electron app, save domain to
settings.yamlas soon as user enters it and hitsLog In. Since electron app does not require login to make requests, this will succeed. -
For the web app,
PLOTLY_API_URLmust be explicitly set as an environment variable.
👍
@tarzzz How about passing PLOTLY_API_URL in oauth callback?
@tarzzz Is this issue still present in Falcon v2.8?
@tarzzz bump :)
Is this issue still present in Falcon v2.8?
Yes, I see it in v2.8 as well.. !!