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

Separate ueberauth config for different app under umbrella project #78

Closed
bluzky opened this Issue May 21, 2018 · 2 comments

Comments

Projects
None yet
3 participants
@bluzky
Copy link

bluzky commented May 21, 2018

Hi! I'm working on an umbrella project, I have 2 apps, one for admin and one for user.
I want both admin and user can login using facebook account.

  • admin -----auth---> facebook app for admin
  • user -----auth----> facebook app for user

this is config for admin app

config :ueberauth, Ueberauth,
  providers: [
    facebook: {Ueberauth.Strategy.Facebook, []}
  ]

config :ueberauth, Ueberauth.Strategy.Facebook.OAuth,
  client_id: System.get_env("ADMIN_FB_ID"),
  client_secret: System.get_env("ADMIN_FB_SECRET")

this is config for user app

config :ueberauth, Ueberauth,
  providers: [
    facebook: {Ueberauth.Strategy.Facebook, []}
  ]

config :ueberauth, Ueberauth.Strategy.Facebook.OAuth,
  client_id: System.get_env("FB_ID"),
  client_secret: System.get_env("FB_SECRET")

Admin account cannot login using facebook because it use configuration for user app.
How can I separate configuration for admin and user app?
Thank you

@scrogson

This comment has been minimized.

Copy link
Member

scrogson commented May 29, 2018

@bluzky unfortunately, what you wish to do is not currently possible. Sorry about that.

Configuration in Elixir is a hot topic at the moment and the configuration story for Ueberauth really needs to be revisited. This will likely require a new major version. There is currently nothing on the road map for this, but the core team and I will discuss it soon.

@yordis yordis added this to the v1 milestone Dec 24, 2018

@yordis

This comment has been minimized.

Copy link
Member

yordis commented Dec 24, 2018

Moved to #90

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment