dwyl / auth

🚪 🔐 UX-focussed Turnkey Authentication Solution for Web Apps/APIs (Documented, Tested & Maintained)
https://authdemo.fly.dev
GNU General Public License v2.0
135 stars 9 forks source link

PR Deploy Fly.io App issue #172 #173

Closed nelsonic closed 2 years ago

nelsonic commented 2 years ago

Deploying a fresh version of the auth App to Fly.io for #172

Relevant reading: https://blog.miguelcoba.com/deploying-an-elixir-release-using-docker-on-flyio thanks to @miguelcoba ⭐

codecov[bot] commented 2 years ago

Codecov Report

Merging #173 (1db1f87) into main (ead7450) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #173   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           24        27    +3     
  Lines          572       586   +14     
=========================================
+ Hits           572       586   +14     
Impacted Files Coverage Δ
lib/auth/app.ex 100.00% <ø> (ø)
lib/auth/init/init.ex 100.00% <100.00%> (ø)
lib/auth/init/roles.ex 100.00% <100.00%> (ø)
lib/auth/init/statuses.ex 100.00% <100.00%> (ø)
lib/auth/person.ex 100.00% <100.00%> (ø)
lib/auth/status.ex 100.00% <100.00%> (ø)
lib/auth_web/controllers/init_controller.ex 100.00% <100.00%> (ø)
lib/auth_web/router.ex 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ead7450...1db1f87. Read the comment docs.

nelsonic commented 2 years ago

Created the necessary environment variables: image

Keep getting interrupted so it's taking a bit longer than it needs to. ⏳

nelsonic commented 2 years ago

on hold until #174 is sorted. ⏳

nelsonic commented 2 years ago

Once #175 has been merged, we should be able to run Auth.Init.main/0 on Fly.io and the deploy should work. 🤞

nelsonic commented 2 years ago

Export the new environment variable:

flyctl secrets set AUTH_URL=authprod.fly.dev
nelsonic commented 2 years ago

Pretty close to having this working on Fly.io: https://authprod.fly.dev/

image

nelsonic commented 2 years ago

Working: https://auth.dwyl.com/people

image

nelsonic commented 2 years ago

@SimonLab apologies for the massive PR. 🙄 I honestly just wanted to deploy to Fly.io ... 🚀 but ended up having to create the init.ex and init_controller.ex in order to achieve the goal. 🎯 Please review and give feedback when you can. 🙏 Once you're happy, please merge.

This is already deployed to https://auth.dwyl.com/ feel free to try and break it! 💭

nelsonic commented 2 years ago

Hi @SimonLab 👋 Comments addressed and changes applied. Thanks! ☀️

nelsonic commented 2 years ago

Thank you @SimonLab ☀️