Skip to content
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

feat: Define dynamic newState function #3

Merged
merged 1 commit into from
Oct 23, 2020

Conversation

asetty
Copy link
Collaborator

@asetty asetty commented Oct 23, 2020

Create state package and relativeURL + schemeAndHost
StateFuncs. The value of SESSION_DOMAIN is used to pick
the strategy. If SESSION_DOMAIN is set then
AuthService will be able to serve requests
for all subdomains of SESSION_DOMAIN.

Also, because createNonce is not used anywhere
else it makes since to move it to this package.

This commit adds 2 new env vars:
2. SCHEME_DEFAULT - default request scheme
3. SCHEME_HEADER - header to get original request scheme from

Signed-off-by: Adam Setters asetty@arista.com

Create state package and relativeURL + schemeAndHost
StateFuncs. The value of SESSION_DOMAIN is used to pick
the strategy. If SESSION_DOMAIN is set then
AuthService will be able to serve requests
for all subdomains of SESSION_DOMAIN.

Also, because createNonce is not used anywhere
else it makes since to move it to this package.

This commit adds 2 new env vars:
2. SCHEME_DEFAULT - default request scheme
3. SCHEME_HEADER - header to get original request scheme from

Signed-off-by: Adam Setters <asetty@arista.com>
@asetty asetty merged commit 663cb8f into aristanetworks:asetty-dev Oct 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant