Bug 1469470 - Company Specific Messages table on the login page should gracefully handle large amounts of text.
Summary: Company Specific Messages table on the login page should gracefully handle la...
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Settings
Version: 6.3.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: Unspecified
Assignee: Tomas Strachota
QA Contact: Nikhil Kathole
URL: http://projects.theforeman.org/issues...
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-07-11 10:32 UTC by Nikhil Kathole
Modified: 2019-08-12 16:36 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-02-21 17:06:59 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
screenshot of UI (499.40 KB, image/png)
2017-07-11 10:32 UTC, Nikhil Kathole
no flags Details
Screenshot of login page (301.91 KB, image/png)
2017-10-30 10:54 UTC, Nikhil Kathole
no flags Details


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 21200 0 Normal Closed Company Specific Messages table on the login page should gracefully handle large amounts of text. 2020-05-18 20:44:54 UTC

Description Nikhil Kathole 2017-07-11 10:32:02 UTC
Created attachment 1296175 [details]
screenshot of UI

Description of problem: The Company Specific Messages on the login page can cover all login page and the user interface under settings-> general tab breaks.Thus, there should be character limit.


Version-Release number of selected component (if applicable): sat 6.3 snap 6.0


How reproducible: always


Steps to Reproduce:
1. Navigate to Administer -> settings
2. Click on general tab 
3. Enter large Login page footer message

Actual results:
Login page and UI under settings-> general breaks.

Expected results:
There should be constraint to character limit so that login page UI and  settings-> general tab page should not break.

Additional info:

Comment 3 Daniel Lobato Garcia 2017-08-09 09:55:07 UTC
I don't think there should be a limit. Instead, the settings should be able to resize the table accordingly to the content. In fact some of the requesters of the RFE https://bugzilla.redhat.com/show_bug.cgi?id=726570 ask for pretty large texts to be included.

Comment 4 Corey Welton 2017-08-11 18:32:37 UTC
Agree, to some degree, with comment #3.  The proper way would be to assure table can gracefully handle a bunch of text.

There probably should be some sort of validation, however, that someone can't DDoS or otherwise make a login screen unusable due to malicious (or inadvertent) inclusion of excessive content.

Comment 5 Walden Raines 2017-10-04 20:36:49 UTC
Created redmine issue http://projects.theforeman.org/issues/21200 from this bug

Comment 6 Satellite Program 2017-10-10 15:12:46 UTC
Upstream bug assigned to tstrachota

Comment 7 Walden Raines 2017-10-17 14:17:47 UTC
PR:  https://github.com/theforeman/foreman/pull/4915

Comment 8 Satellite Program 2017-10-23 14:13:33 UTC
Moving this bug to POST for triage into Satellite 6 since the upstream issue http://projects.theforeman.org/issues/21200 has been resolved.

Comment 9 Nikhil Kathole 2017-10-30 10:52:27 UTC
VERIFIED

Version Tested:
Satellite-6.3 Snap 22

steps:

1. Inserted large text in Login page footer message parameter under settings

Login page handled the larger data.

Comment 10 Nikhil Kathole 2017-10-30 10:54:19 UTC
Created attachment 1345372 [details]
Screenshot of login page

Comment 11 Bryan Kearney 2018-02-21 17:06:59 UTC
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA.

For information on the advisory, and where to find the updated files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2018:0336


Note You need to log in before you can comment on or make changes to this bug.