-
Notifications
You must be signed in to change notification settings - Fork 15
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
Unexpected server process output when trying to delete an entry #22
Comments
I found out that the error disappears when I remove the |
Sorry to hear you're having trouble. Thanks for the minimal reproduction script! Unfortunately, this looks like a problem somewhere in ApacheDS. Ladle uses standard out as a control channel with its ApacheDS runner and expects only messages from the runner to appear there. It looks like in this situation, ApacheDS itself is printing something to standard out directly (rather than using the normal logging mechanisms, which ladle's runner captures). I tried updating to the latest ApacheDS, but the problem remains. Your monkey patch looks fine if it works for you, but I don't want to do a similar thing in Ladle because it might mask other bugs. The permanent fix would be to track down where this spurious log message is coming from in ApacheDS and block it. |
Thanks for your quick reply. My monkey patch should not be considered production ready, otherwise I would have made it into a PR 😉. If I understand this correctly, someone forgot to remove a debug statement from the ApacheDS source. I agree this should be fixed in ApacheDS, not in Ladle. |
I think the message is printed in one of these methods: Maybe that helps. |
Good find. It looks like the message is specifically printed during delete here: |
Code to reproduce:
I could bypass the error message by changing
Ladle::Server::ApacheDSController#watch
to only report an error when the line starts with an all uppercase word. Then, the deletion is actually performed successfully. However, this seems like an ugly hack. I am not an LDAP expert, maybe I am doing something else wrong. Any help on this would be much appreciated!The text was updated successfully, but these errors were encountered: