Project

General

Profile

Bug #36019

Pass URL parameters to foreman_url as hash

Added by Leos Stejskal 2 months ago. Updated 16 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Unattended installations
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

Some templates add URL params to the foreman_url like this:
<%= foreman_url('provision') + "&static=yes" %>

That cause issues when token_duration setting is set to 0,
generating URL '.../unattended/provision&static=yes'
instead of '.../unattended/provision?static=yes'

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

Associated revisions

Revision 95196091 (diff)
Added by Leos Stejskal about 2 months ago

Fixes #36019 - Pass URL params to foreman_url as hash

Some templates add URL params to the foreman_url like this:
<%= foreman_url('provision') + "&static=yes" %>

That cause issues when token_duration setting is set to 0,
generating URL '.../unattended/provision&static=yes'
instead of '.../unattended/provision?static=yes'

History

#1 Updated by Leos Stejskal 2 months ago

  • Assignee changed from Lukas Zapletal to Leos Stejskal
  • Category set to Templates
  • Subject changed from Pass URL parameters to foreman_url as hash to Pass URL parameters to foreman_url as hash

#2 Updated by The Foreman Bot 2 months ago

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

#3 Updated by Ewoud Kohl van Wijngaarden 2 months ago

  • Target version set to 3.5.2

#4 Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases 3.6.0 added

#5 Updated by Leos Stejskal about 2 months ago

  • Status changed from Ready For Testing to Closed

#6 Updated by Ewoud Kohl van Wijngaarden about 1 month ago

  • Triaged changed from No to Yes
  • Category changed from Templates to Unattended installations

#7 Updated by The Foreman Bot 20 days ago

  • Fixed in Releases 3.5.2 added

#8 Updated by The Foreman Bot 16 days ago

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

#9 Updated by Ewoud Kohl van Wijngaarden 16 days ago

  • Pull request deleted (https://github.com/theforeman/foreman/pull/9614)

Also available in: Atom PDF