389-ds-base

#51062 Issue 51054 - Revise ACI target syntax checking
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base issue51054  into  master

Download 51062.patch

Bug Description:

The previous commit enforced a strict syntax that was previously allowed. This is causing regressions for customers and community members.

Fix Description:

Reject ACI's that use more than one equal sign between the target keyword and the value, but do not enforce that the values are quoted. A flag was added that we can turn on strict syntax at a later date, but for now we will continue allow values without quotes.

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

LGTM! Ack

rebased onto 916d13bc230f5edffd8f4fb9da0a30fcde3eb573

Pull-Request has been merged by mreynolds

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/4115

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
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.