-
Notifications
You must be signed in to change notification settings - Fork 1
Deploy Staging App to Heroku? #14
Comments
I've added Postgres, the Elixir Build Packs and the |
This looks good, this should work well for testing as our firmware clients can't change the state of the Hub server, so we should be okay for running parallel tests etc. |
@th0mas cool. thanks for confirming. I will proceed with deploying the app then. 👍 |
Had to update From: smart-home-auth-server/config/dev.exs Lines 4 to 8 in 67f9883
To: # Configure your database
config :smart_home_auth, SmartHomeAuth.Repo,
username: "postgres",
password: "postgres",
database: "smart_home_auth_dev",
hostname: "localhost", As per the "Run" instructions: https://github.com/dwyl/smart-home-auth-server/tree/67f9883c6e21c50e8bf334018cfca42b27295474#run |
https://smart-home-auth-server.herokuapp.com However after authenticating with my Google Account I see the following 500 Error Page:
|
@th0mas when you're back from time off. It would be good to pair on this. 💭 |
Ran the migration on Heroku:
Restarted the App. Boom. Works. |
@th0mas please share the email address you use for Heroku on Signal so I can make you a collaborator |
App deployed to Heroku and made accessible to collaborators, closing 👍 |
No logout implemented ... let me remove session manually. Confirmed login with GitHub failing: Uses https://github.com/dwyl/elixir-auth-github under the hood. Ok, now checking if the So I think the issue is that GitHub Auth is not passing through the email address. |
@nelsonic is it safe to say that the issue above is an |
@th0mas yeah, it's definitely an |
The possible future issue by using the Github username when the email is not available is that the username can be changed in Github: https://docs.github.com/en/enterprise/2.14/user/articles/changing-your-github-username At the moment the email is used to retreive or create the person: So using the username when the email is not defined could possibly allow another person to access information of a user who has recently updated her username. I suggest that instead of allowing people to login/register with Github when the email is hidden we display an error message explaining that an email is required. |
@SimonLab Yeah, requiring email when their GitHub account does not return one could be a better option. |
As noted in dwyl/smart-home-firmware#15 (comment) I think it might be more effective to run the "Hub" Phoenix App on Heroku instead of trying to run it on CI. By auto-deploying the latest
master
version to Heroku it's always current and we can debug things independently.I have created a brand new (separate) Heroku Account so that the App gets the full 550 "free dyno hours" and we never have to worry about it being unavailable/offline. dwyl/email#38 | dwyl/auth#88
#LessonLearned
🙄Let me know your thoughts on this, I don't mind doing the setup and will share the access to the app once it's working.
The text was updated successfully, but these errors were encountered: