Project

General

Profile

Actions

Bug #35385

closed

With every edit of an exising webhook, the value in password field disappears in Satellite 6.10/6.11/6.12

Added by Oleh Fedorenko almost 2 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Target version:
-
Difficulty:
Triaged:
No
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=2119155

Description of problem:

The password field of a webhook is not persistent while other parameters in the webhook is being modified and submitted

Version-Release number of selected component (if applicable):

Satellite 6.10
Satellite 6.11
Satellite 6.12 ( internal latest snap )

How reproducible:

100%

Steps to Reproduce:
1. Install any of the versions of the satellite as mentioned before

2. Go to Administer --> Webhooks and Create a webhook by selecting any arbitrary stuff But make sure to go to the Credentials page and mention a username and password in their respective fields

3. Open CLI and go inside rake console to inspect the data.

  1. foreman-rake console

Webhook.last.password

Webhook.last.user

4. Go back to Administer --> Webhooks , simply edit the webhook, and then submit. We don't even need to make any change.

5. Come back inside CLI where rake console is open and type

Webhook.last.password

Webhook.last.user

Actual results:

At Step 3: We can see the user and password is set:

irb(main):001:0> Webhook.last.password
=> "redhat"
irb(main):002:0> Webhook.last.user
=> "admin"

At Step 5, We see the user is still set but password has become blank

irb(main):003:0> Webhook.last.password
=> ""

irb(main):004:0> Webhook.last.user
=> "admin"

Even before submitting the hook at Step 4, if we go to the Credentials tab, we will get to see the same thing i.e. password field is blank as soon as we edit the Webhook.

Expected results:

The password should remain set and intact unless manually modified or nullified.

Additional info:

NA

Actions #1

Updated by Oleh Fedorenko almost 2 years ago

  • Project changed from Foreman to Webhooks
  • Assignee changed from Lukas Zapletal to Oleh Fedorenko
Actions #2

Updated by Oleh Fedorenko almost 2 years ago

  • Status changed from New to Assigned
Actions #3

Updated by The Foreman Bot almost 2 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_webhooks/pull/54 added
Actions #4

Updated by Oleh Fedorenko almost 2 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF