Project

General

Profile

Bug #30684

HTTP proxy password got reset to empty string after editing can cause 407 "Proxy Authentication Required" error

Added by Avi Sharvit 5 months ago. Updated 5 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Network
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

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

Description of problem:
HTTP proxy password will reset to empty string after editing the existing HTTP proxy in the Web UI.

Before:
#<ActiveRecord::Relation [#<HttpProxy id: 5, name: "test_proxy", url: "http://xx.xx.xx.xx:xxxx", username: "myname", password: "encrypted-blah blah blah">]>

After:
#<ActiveRecord::Relation [#<HttpProxy id: 5, name: "test_proxy", url: "http://xx.xx.xx.xx:xxxx", username: "myname", password: "">]>

Steps to Reproduce:
1. Create a http proxy with username and password
2. Add it as the global content proxy.
3. Ensure it is working by enabling a repository or doing cdn scans.
4. Edit the http proxy you just created, don't do anything just click save button.
5. Try to enable a repository again using hammer

Actual results:

hammer repository-set enable --name "Red Hat Satellite Tools 6.7 for RHEL 8 x86_64 (RPMs)" --organization redhat
Could not enable repository:
407 "Proxy Authentication Required"

Cdn scan in Web UI -> Red Hat Repositories gets the following error:

No repositories available error.

Expected results:
Able list available repositories
Able enable repository successfully

Associated revisions

Revision 29dced09 (diff)
Added by Avi Sharvit 5 months ago

Fixes #30684 - add password edit button to the http proxy form

It should allow users to send the form without editing the password

History

#1 Updated by The Foreman Bot 5 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/7917 added

#2 Updated by The Foreman Bot 5 months ago

  • Fixed in Releases 2.3.0 added

#3 Updated by Avi Sharvit 5 months ago

  • Status changed from Ready For Testing to Closed

#4 Updated by The Foreman Bot 5 months ago

  • Pull request https://github.com/theforeman/foreman/pull/7975 added

Also available in: Atom PDF