Project

General

Profile

Actions

Bug #29202

closed

Disallow commas in activation key names

Added by Jonathon Turel about 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Low
Category:
Activation Key
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

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

++ This bug was initially created as a clone of Bug #1657076 ++

Description of problem:
subscription manager register activationkey with special character failed

How reproducible:
always

Steps to Reproduce:
1.Create a activationkey named ",.;?" on satellite6.4
Please reference picture
activationkey.png

2.Register satellite server with this activationkey
  1. subscription-manager register --org=Default_Organization --activationkey=',.;?'
    HTTP error (404 - Not Found): Couldn't find activation key ''

3. Register satellite server with cockpit
Error:
HTTP error (404 - Not Found): Couldn't find activation key ''
Please reference picture
cockpit_activationkey.png

4. Used other special characters, it work well.
  1. subscription-manager register --org=Default_Organization --activationkey='!@#$%^&*()_+-={}{}[<.>'
    The system has been registered with ID: 5978380b-9ba8-40aa-9d66-4f2641cfa403
    The registered system name is: dell-r730-001-guest14.dsal.lab.eng.rdu2.redhat.com
    Installed Product Current Status:
    Product Name: Red Hat Enterprise Linux for x86_64 High Touch Beta
    Status: Subscribed

Actual results:
As step 3

Expected results:
As satellite server support special character creation, register to satellite server also should support, even though we do not use this characters usually.
If we think these characters are not useful, maybe satellite server need to add validation for creating activate key using these special characters.

Actions #1

Updated by Jonathon Turel about 4 years ago

  • Subject changed from Disallow commas from activation key names to Disallow commas from activation key names
  • Target version set to Katello 3.16.0
Actions #2

Updated by Jonathon Turel about 4 years ago

  • Subject changed from Disallow commas from activation key names to Disallow commas in activation key names
Actions #3

Updated by The Foreman Bot about 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/Katello/katello/pull/8589 added
Actions #4

Updated by The Foreman Bot about 4 years ago

  • Fixed in Releases Katello 3.16.0 added
Actions #5

Updated by Jonathon Turel about 4 years ago

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

Updated by Ian Ballou about 4 years ago

  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF