Bug 1992663

Summary:	[RFE]: Include example URL on tooltip for "Target URL" on create webhook page
Product:	Red Hat Satellite	Reporter:	Taft Sanders <tasander>
Component:	Hooks and Webhooks	Assignee:	Oleh Fedorenko <ofedoren>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Lukáš Hellebrandt <lhellebr>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	6.10.0	CC:	aruzicka, lhellebr, mhulan, ofedoren, rlavi
Target Milestone:	Unspecified	Keywords:	EasyFix, FutureFeature
Target Release:	Unused
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:	rubygem-foreman_webhooks-3.2.0	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2024-03-14 12:45:20 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Taft Sanders 2021-08-11 13:55:49 UTC

Description of problem:
To illustrate the needed components of the URI, it would be helpful to a user to see an example in the tooltip.

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

How reproducible:
n/a

Steps to Reproduce:
1.
2.
3.

Actual results:
Tooltip shows:
Target URL that should be called by Satellite

Expected results:
Tooltip should show:
Target URL that should be called by Satellite. 
Example: https://my.webhook.com/1234-5678-90ab-cdef

Additional info:
This will help avoid simple errors or forgetting to add the HTTP schema to the URL.

Comment 1 Brad Buckingham 2022-09-02 20:25:18 UTC

Upon review of our valid but aging backlog the Satellite Team has concluded that this Bugzilla does not meet the criteria for a resolution in the near term, and are planning to close in a month. This message may be a repeat of a previous update and the bug is again being considered to be closed. If you have any concerns about this, please contact your Red Hat Account team.  Thank you.

Comment 2 Brad Buckingham 2022-09-05 22:55:19 UTC

Upon review of our valid but aging backlog the Satellite Team has concluded that this Bugzilla does not meet the criteria for a resolution in the near term, and are planning to close in a month. This message may be a repeat of a previous update and the bug is again being considered to be closed. If you have any concerns about this, please contact your Red Hat Account team.  Thank you.

Comment 4 Oleh Fedorenko 2023-03-02 14:12:56 UTC

Created redmine issue https://projects.theforeman.org/issues/36155 from this bug

Comment 8 Lukáš Hellebrandt 2023-07-25 14:04:44 UTC

Verified with Sat 6.14.0 snap 8.0.

In WebUI: Administer -> Webhooks -> Create new
The "i" icon next to "Target URL" shows, upon clicking:
"""
Target URL that should be called by Satellite (ERB allowed).
https://host.example.com/inventory/<%= @object.id %>
"""

This correctly shows all the URL components EXCEPT a port. However, for non-standard ports (i.e. other than 80 and 443), the Satellite is not expected to run out of the box - it needs Selinux reconfigured or disabled. Therefore, I think this is the most appropriate tooltip text. It even contains an example of variable usage directly in URL.

Note that using a variable with "Test Webhook" functionality raises an uncaught exception, I will report this as a standalone BZ.

Comment 10 Adam Ruzicka 2024-03-14 12:45:20 UTC

Per #8 this was verified in 6.14.0 which was released some time ago. Closing.