joshuataylor / ueberauth_esi

Ueberauth authentication for ESI, for EVE Online
MIT License
0 stars 1 forks source link

Declares to implement OAuth v2 but using v1 #1

Open blackmadness opened 4 years ago

blackmadness commented 4 years ago

According to the source, using this:

  @defaults [
    strategy: __MODULE__,
    site: "https://login.eveonline.com",
    authorize_url: "https://login.eveonline.com/oauth/authorize",
    token_url: "https://login.eveonline.com/oauth/token",
  ]

means the strategy using OAuth v1. but the docs states it is using v2.

As you can see here, point 3, urls should have v2 token into them.

joshuataylor commented 4 years ago

I think when I set this up, the URL was v2? Or maybe it was v1?

I'll try with v2 and see what happens.

Cheers!

joshuataylor commented 4 years ago

So the ESI API has two options when creating a key:

  1. Auth only
  2. Auth+API

I'm testing with both, and you don't need to pass in a scope when using #1, so thinking about making it so you can pass in the scopes you need but by default making this nil.