PR#51215: Issue 51086 - Fix instance name length for interactive install - 389-ds-base

Simon Pichugin • 3 years ago

d5c9c4e

file modified

		`@@ -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.

Reviewed by: ?

Ack

rebased onto d5c9c4e

3 years ago

Pull-Request has been merged by spichugi

3 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

3 years ago

Assignee

None

Tags

No Tags