Project

General

Profile

Actions

Bug #38195

closed

Template sync doesn't use proxy when ssh used

Added by Adam Lazik about 2 months ago. Updated 1 day ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
-
Fixed in Releases:
Found in Releases:

Description

Description of problem:

There is a new feature that allows template sync to use proxy when syncing templates. It's possible to sync templates using http or ssh (to/from git) but only http actually uses the proxy. The ssh connection goes directly. This is not documented. It's possible to specify a proxy even when ssh address is entered and the proxy is then silently ignored.

How reproducible:

Deterministic

Is this issue a regression from an earlier version:

Issue in a new feature

Steps to Reproduce:

1. In WebUI, go to Hosts -> Templates -> Sync Templates

2. As "Repo", specify a git ssh URI such as `ssh::port/user/reponame`

3. Specify other necessary settings, such as correct branch

4. Set HTTP proxy policy either to global, or to custom (and specify the custom HTTP proxy)

5. Submit

Actual behavior:

Proxy is not used

Expected behavior:

Either proxy should be used, or it should be documented that proxy can't be used for ssh and the UI (Web, Hammer, API) shouldn't allow to specify it together with ssh URI.


Related issues 1 (0 open1 closed)

Precedes Templates - Bug #38311: Redundant whitespaces in http_proxy_policy param descriptionClosedOleh FedorenkoActions
Actions #1

Updated by The Foreman Bot about 2 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman_templates/pull/194 added
Actions #2

Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases foreman_templates 10.1.0 added
Actions #3

Updated by Adam Lazik about 2 months ago

  • Status changed from Ready For Testing to Closed
Actions #4

Updated by Adam Ruzicka about 2 months ago

  • Fixed in Releases foreman_templates 10.0.4 added
  • Fixed in Releases deleted (foreman_templates 10.1.0)
Actions #5

Updated by Oleh Fedorenko 9 days ago

  • Precedes Bug #38311: Redundant whitespaces in http_proxy_policy param description added
Actions #6

Updated by The Foreman Bot 1 day ago

  • Pull request https://github.com/theforeman/foreman_templates/pull/200 added
Actions

Also available in: Atom PDF