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

SSL verify peer checkbox wants to be always checked #75

Closed
pdzionek opened this issue Apr 29, 2019 · 0 comments
Closed

SSL verify peer checkbox wants to be always checked #75

pdzionek opened this issue Apr 29, 2019 · 0 comments
Labels
bug Something isn't working
Milestone

Comments

@pdzionek
Copy link

Describe the bug
SSL verify check box is always checked, even though it shouldn't. It happens in two cases, when you add a new node and you uncheck the box to run "Test Connection" it checks it by itself, which is really annoying. You have to uncheck it again, before adding the node. And another case is when you edit the resource, which was created with unchecked ssl verify. Every time, you edit it is always checked, so you have to remember to uncheck it before submitting changes.

To Reproduce

Steps to reproduce the behavior in Create Compute Resource:

  1. Go to 'Infrastructure -> Compute Resource '
  2. Click on 'Create Compute resource'
  3. Choose Proxmox provider
  4. Type in the proper url and login credentials.
  5. Uncheck "SSL Verify peer"
  6. Click "Test Failed" button
  7. "SSL verify peer" is checked again

Or

Steps to reproduce the behavior in Edit Compute Resource:

  1. Go to 'Infrastructure -> Compute Resource '
  2. Click on 'Your proxmox node resource'
  3. Click 'Edit button'
  4. "SSL verify peer" is checked, even when it should not be.

Expected behavior
"SSL verify peer" should be consistent with data provided by the user. If you create a compute resource with disabled ssl verify peer, it should be in that state in Edit screen. Also it should beave

Foreman - Proxmox versions
foreman 1.21.3-1
ruby-foreman-fog-proxmox 0.6.0-1
Proxmox cluster 5.3.7

@pdzionek pdzionek changed the title SSL verify peer wants to be always checked SSL verify peer checkbox wants to be always checked Apr 29, 2019
@tristanrobert tristanrobert added the bug Something isn't working label Apr 29, 2019
@tristanrobert tristanrobert added this to the 0.8.0 milestone Apr 29, 2019
tristanrobert added a commit that referenced this issue Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants