Bug 1939703 - [AI-UI-Wizard] Unlimited number of characters accepted for base domain name
Summary: [AI-UI-Wizard] Unlimited number of characters accepted for base domain name
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: assisted-installer
Version: 4.7
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: Marek Libra
QA Contact: Omri Hochman
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-03-16 20:54 UTC by Steve Kramling
Modified: 2022-08-28 08:45 UTC (History)
4 users (show)

Fixed In Version: ocp-metal-ui-v1.5.15
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1939873 (view as bug list)
Environment:
Last Closed: 2022-08-28 08:45:59 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
large base domain name (263.57 KB, image/png)
2021-03-16 20:54 UTC, Steve Kramling 		no flags Details
base domain install failure (478.65 KB, image/jpeg)
2021-03-17 14:02 UTC, Steve Kramling 		no flags Details
View All


Links
System ID Private Priority Status Summary Last Updated
Github openshift-assisted assisted-ui-lib pull 546 0 None open BZ 1939703: Add validation for domain name length 2021-04-13 12:55:38 UTC

Description Steve Kramling 2021-03-16 20:54:15 UTC 
Created attachment 1763780 [details]
large base domain name

Created attachment 1763780 [details]
large base domain name

Description of problem:
[AI-UI-Wizard] Unlimited number of characters accepted for base domain name

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

How reproducible:
100%

Steps to Reproduce:
1.Configure cluster details
2.Configure base domain with large number of characters
3.

Actual results:
base domain accepted as valid

Expected results:
some error correction for max number of characters 

Additional info:
In this example I used the following base domain name which was accepted:

ocp-edge-cluster-sealusa48.qe.lab.redhat.commmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmmm
Comment 1 Steve Kramling 2021-03-17 14:02:45 UTC 
Created attachment 1764055 [details]
base domain install failure

The long base domain name prevents a successful installation.  See attachment
Comment 2 Jiri Tomasek 2021-03-18 15:20:23 UTC 
"Domain names (including the names of domains, hosted zones, and records) consist of a series of labels separated by dots. Each label can be up to 63 bytes long. The total length of a domain name cannot exceed 255 bytes, including the dots."
Comment 3 Marek Libra 2021-04-13 12:58:46 UTC 
Ronnie, What about implementing similar input validation on the backend as well?
Comment 4 Ronnie Lazar 2021-04-13 13:32:07 UTC 
That is handelled in https://bugzilla.redhat.com/show_bug.cgi?id=1939873
Comment 5 Steve Kramling 2021-04-27 15:58:13 UTC 
Verified in 1.5.16
Note You need to log in before you can comment on or make changes to this bug.