search

karnthis / esiil

ESI API Interface
4 stars 2 forks source link

buildRequestURL should accept a state value #2

Closed justindh closed 4 years ago

justindh commented 5 years ago

maybe something like this

buildRequestURL(scopes = [], state = '') { const useScope = (this.masterScopes.length > 0) ? this.masterScopes : scopes if (useScope.length == 0) throw new Error('must have at least 1 scope selected') return [https://login.eveonline.com/oauth/authorize/?response_type=code, redirect_uri=${encodeURI(this.callbackURL)}, client_id=${this.clientID}, scope=${useScope.join(' ')}, state=${state}].join('&') }

karnthis commented 5 years ago

This is a good suggestion. I don't personally use state, so it did not occur to me to include it.

karnthis commented 4 years ago

Now available on the Alpha branch

justindh commented 4 years ago

Works :)

karnthis commented 4 years ago

Now available in v0.1.2, closing issue