Page MenuHomePhorge

[Python 3] fork code harmonizing

Authored by ghane on Mar 14 2022, 10:41 AM.
Referenced Files
Unknown Object (File)
Sat, Sep 30, 6:43 PM
Unknown Object (File)
Sat, Sep 30, 6:43 PM
Unknown Object (File)
Fri, Sep 29, 10:41 AM
Unknown Object (File)
Fri, Sep 29, 4:42 AM
Unknown Object (File)
Sat, Sep 23, 7:05 AM
Unknown Object (File)
Thu, Sep 7, 1:58 AM
Unknown Object (File)
Wed, Sep 6, 8:41 PM
Unknown Object (File)
Tue, Sep 5, 5:47 PM



fixes error in Python3 running in fork mode

Code harmonized with code in def run of kolabd/ and wallace/

Diff Detail

rP pykolab
Lint Not Applicable
Tests Not Applicable

Event Timeline

ghane created this revision.
sicherha subscribed.

Could you add to the summary of this differential a description of the concrete error that is being solved here? I presume that some part of Python (which one?) complained about one of the file descriptors for stdin/ stdout/ stderr being unavailable, but it would be great to make that explicit.

This revision now requires changes to proceed.Mar 14 2022, 11:51 PM

with --fork

testsaslauthd -u user(@domain) -p password
0: NO "authentication failed"

  • Logging error ---

Traceback (most recent call last):

File "/usr/lib/python3.x/logging/", line 1037, in emit
  stream.write(msg + self.terminator)

BrokenPipeError: [Errno 32] Broken pipe
Call stack:

File "./", line 43, in <module>

Message: "Authentication for 'user(@domain)' succeeded"

I tried to localize the error on other parts of the code, but I couldn't localize.
without --fork the code worked.

testsaslauthd -u user(@domain) -p password
0: OK "Success."

I just checked against the same fork code snippet on wallace and kolabd, C&P the missing code part and it worked.

So I just harmonized the fork code in def run on the daemon.

This revision is now accepted and ready to land.Mar 16 2022, 10:28 PM
This revision was automatically updated to reflect the committed changes.