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.

Sign up for GitHub

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

Jump to bottom

Make code more secure when using remote clusters #104

Closed
ifireball opened this issue Oct 31, 2018 · 2 comments
Closed

Make code more secure when using remote clusters #104

ifireball opened this issue Oct 31, 2018 · 2 comments

Comments

@ifireball
Copy link
Contributor

Right now when using remote OpenShift clusters the code here will by default:

  1. Disable all TLS checking
  2. Require one to specify the OpenShift username and password on the command line

We should:

  1. Enable TLS checking by default and have a parameter to disable it
  2. Enable using a token instead of a password
  3. Enable assuming the local user had already logged into the OpenShift cluster.
@arilivigni
Copy link
Member

Maybe a nice to have not really critical

ifireball added a commit to ifireball/contra-env-setup that referenced this issue Nov 5, 2018
@ifireball
Alternative OpenShift login methods
b320696 
Add support for alternative ways to login to OpenShift:
1. Support skipping the loging process and assuming the user logged-in
   manually via the command line
2. Support logging in with a token instead of a username and a password

This partially resolves issue CentOS-PaaS-SIG#104

Signed-off-by: Barak Korren <[email protected]>
@ifireball ifireball mentioned this issue Nov 5, 2018
Merged
ifireball added a commit to ifireball/contra-env-setup that referenced this issue Nov 5, 2018
@ifireball
Alternative OpenShift login methods
8a01327 
Add support for alternative ways to login to OpenShift:
1. Support skipping the loging process and assuming the user logged-in
   manually via the command line
2. Support logging in with a token instead of a username and a password

This partially resolves issue CentOS-PaaS-SIG#104

Signed-off-by: Barak Korren <[email protected]>
@arilivigni
Copy link
Member

This fix was merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ifireball @arilivigni