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

blue/green deploy means that GitHub OAuth key/secret invalid #92

Open
rwinch opened this issue May 11, 2016 · 5 comments
Open

blue/green deploy means that GitHub OAuth key/secret invalid #92

rwinch opened this issue May 11, 2016 · 5 comments
Assignees

Comments

@rwinch
Copy link
Collaborator

rwinch commented May 11, 2016

GitHub requires we map the key/secret to single URL. This means the specific urls (i.e. pivotal-cla-production-blue.cfapps.io) will not be able to authenticate.

Ideally we would support this use case.

@rwinch rwinch added this to the 1.0.0 milestone May 11, 2016
@mp911de
Copy link
Collaborator

mp911de commented May 24, 2016

We'll have to create Github applications for dev and staging. Going to ignore the issue we can't login to blue/green domains. Login is only possible using the external URL.

@mp911de mp911de assigned rwinch and unassigned mp911de Jun 7, 2016
@mp911de
Copy link
Collaborator

mp911de commented Jun 7, 2016

@rwinch Could you create a staging application? I'd then adjust the deployment script to accept -Pproduction.oauth.clientId= and -Pstaging.oauth.clientId= so login is possible on staging and production environments.

@rwinch
Copy link
Collaborator Author

rwinch commented Jun 7, 2016

@mp911de We already have a staging app, but it is probably better called smoke test application. We deploy a new instance and wipe the database to run full integration tests against it. See https://github.com/pivotalsoftware/pivotal-cla/blob/b91221105fd0a464ad471561aadb5dda14527eb6/staging.gradle Perhaps we should change the name

Ultimately, I don't think we should spend time on this until we have a problem. We can always manually change the keys if we have an immediate need.

@mp911de
Copy link
Collaborator

mp911de commented Jun 8, 2016

Ok, then let's close this ticket and revisit it later once we run into trouble.

@mp911de mp911de closed this as completed Jun 8, 2016
@rwinch rwinch removed this from the 1.0.0 milestone Jun 8, 2016
@rwinch
Copy link
Collaborator Author

rwinch commented Jun 8, 2016

I'm going to reopen since we know it is an issue. To me this is like an issue that a user reports that may impact one user, but not all users. This issue just doesn't impact us, so we won't prioritize time to it.

@rwinch rwinch reopened this Jun 8, 2016
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

No branches or pull requests

2 participants