supervisor4/python3

[nunojpg]

Under latest versions I found 3 issues:

• if processes output non-ASCII output then the event RESULT message sends the wrong length.
• event_handler response is byte and not string, so all the code fails.
• empty lines are printed to log

I'm not submitting a pull request since they are not being processed. A working version for me follows:

from __future__ import print_function
import sys

def write_stdout(s):
    sys.stdout.write(s)
    sys.stdout.flush()

def write_stderr(s):
    sys.stderr.write(s)
    sys.stderr.flush()

def main():
    while 1:
        write_stdout('READY\n') # transition from ACKNOWLEDGED to READY
        line = sys.stdin.readline()  # read header line from stdin
        headers = dict([ x.split(':') for x in line.split() ])
        data = sys.stdin.read(int(headers['len'])) # read the event payload
        write_stdout('RESULT %s\n%s'%(len(data.encode('utf-8')), data)) # transition from READY to ACKNOWLEDGED

def event_handler(event, response):
    line, data = response.split(b'\n', 1)
    headers = dict([ x.split(b':') for x in line.split() ])
    lines = data.split(b'\n')
    prefix = b'%s %s | '%(headers[b'processname'], headers[b'channel'])
    print(b'\n'.join([ prefix + l for l in lines if l]).decode('utf-8'))

if __name__ == '__main__':
    main()

[twillis]

thanks for this.

[zultron]

This worked for me. Thanks!

[whereisaaron]

I tried this with supervisor 4.2.5 and python 3.11.2 and to didn't work for some reason.
I haven't dug in to work out why yet, not sure how to extract the underlying error message.

It looks like #18 and #20, which have been merged, are alternative fixes for this same problems? #18 decodes the log line before formatting it. #20 encodes the data in the main loop. So this patch may no longer be necessary?

2024-06-08 03:43:17,601 WARN stdout: event caused an error
2024-06-08 03:43:17,601 WARN stdout: has entered the UNKNOWN state and will no longer receive events, this usually indicates the process violated the eventlistener protocol