Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create instance dialog should validate input better #4171

Closed
389-ds-bot opened this issue Sep 13, 2020 · 5 comments
Closed

Create instance dialog should validate input better #4171

389-ds-bot opened this issue Sep 13, 2020 · 5 comments
Labels
closed: fixed Migration flag - Issue cockpit Cockpit UI Plugin
Milestone

Comments

@389-ds-bot
Copy link

Cloned from Pagure issue: https://pagure.io/389-ds-base/issue/51118


Issue Description

We need to valid the root DN, instance name, ports, etc.

Using instance name:

TestSLAPD_%#ls

I get:

Error: '%' must be followed by '%' or '(', found: '%#ls'

Without percent (TestSLAPD_#ls) instance creation starts, but then fails:

When I'm trying port number -1, I get:
Error: Port must be a number between 1 and 65534!
Although it accepts 65535 (as it should).

non-ASCII input is accepted for Directory Manager DN

@389-ds-bot 389-ds-bot added closed: fixed Migration flag - Issue cockpit Cockpit UI Plugin labels Sep 13, 2020
@389-ds-bot 389-ds-bot added this to the 1.4.2 milestone Sep 13, 2020
@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2020-05-27 22:26:22

Metadata Update from @mreynolds389:

@389-ds-bot
Copy link
Author

389-ds-bot commented Sep 13, 2020

Comment from mreynolds (@mreynolds389) at 2020-05-29 21:00:34

#4180

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2020-06-02 16:29:30

Commit a0113b19 relates to this ticket

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2020-06-02 16:47:29

Commit a0113b1 relates to this ticket

3eb8617..b3315a5 389-ds-base-1.4.3 -> 389-ds-base-1.4.3

5ac5b02..db11e8e 389-ds-base-1.4.2 -> 389-ds-base-1.4.2

@389-ds-bot
Copy link
Author

Comment from mreynolds (@mreynolds389) at 2020-06-02 16:47:29

Metadata Update from @mreynolds389:

  • Issue close_status updated to: fixed
  • Issue status updated to: Closed (was: Open)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed: fixed Migration flag - Issue cockpit Cockpit UI Plugin
Projects
None yet
Development

No branches or pull requests

1 participant