Description:
The asterisk character was missing from the allowed character list. Also cleaned up the source in the C file.
Thanks @yrro for contributing the original patch!
relates: https://pagure.io/389-ds-base/issue/50800
Ack from me, I wanted a test case, but you know what, yolo commit. :)
There is a test case, two in fact, but they are missing from the commit - ahhhh!!! I'll need to rebase here in a bit
rebased onto 7dbfc281d51ea8422c8f0c42ed48b2b4ef76cda0
Ack from me, I wanted a test case, but you know what, yolo commit. :) There is a test case, two in fact, but they are missing from the commit - ahhhh!!! I'll need to rebase here in a bit
Test case added, please review...
Thanks mate, ack from me, and also thanks to @yrro for the original - sorry it took so long, but it's working now!
rebased onto c4befd634828338e98b1a0c93db12b1c9e70525c
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/4018
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
Description:
The asterisk character was missing from the allowed character list. Also cleaned up the source in the C file.
Thanks @yrro for contributing the original patch!
relates: https://pagure.io/389-ds-base/issue/50800