#51215 Issue 51086 - Fix instance name length for interactive install
Closed 2 years ago by spichugi. Opened 2 years ago by spichugi.
spichugi/389-ds-base i51086  into  master

@@ -309,6 +309,9 @@ 

  

              val = input('\nEnter the instance name [{}]: '.format(slapd['instance_name'])).rstrip()

              if val != "":

+                 if len(val) > 80:

+                     print("Server identifier should not be longer than 80 symbols")

+                     continue

                  if not all(ord(c) < 128 for c in val):

                      print("Server identifier can not contain non ascii characters")

                      continue

Description: Instance name lenght is not properly validated
during interactive install. Add a check during a user input.

https://pagure.io/389-ds-base/issue/51086

Reviewed by: ?

rebased onto d5c9c4e

2 years ago

Pull-Request has been merged by spichugi

2 years ago

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/4268

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

2 years ago
Metadata