Empty arguments in log messages

[michael-o]

I see constantly empty arguments in log output:

2026-02-14 22:24:13.529998 [pid: 90366, tid: 35041079296] [wsgi:info] Attach interpreter ''.
2026-02-14 22:30:57.214328 [pid: 91649, tid: 35041079296] [wsgi:info] Cleanup interpreter ''.

It is not clear whether this is a bug or the supposed argument can be empty, if it can the ap_log_error() should be guarded and not pass any argument at all as this causes confusion.

[michael-o]

It seems to come from newInterpreterObject() and name can be empty, so can be self->name in (Interpreter_dealloc()), I guess. So there are separate places in this function which can log "''".

[GrahamDumpleton]

It is perfectly fine. An empty string for interpreter name means the main Python interpreter context, as opposed to a sub interpreter context. This is mapped to empty string from when you say %{GLOBAL} in the Apache config if overriding application group. When logged it should always be with some prefixing message such as "Attach interpreter".

[michael-o]

It is perfectly fine. An empty string for interpreter name means the main Python interpreter context, as opposed to a sub interpreter context. This is mapped to empty string from when you say %{GLOBAL} in the Apache config if overriding application group. When logged it should always be with some prefixing message such as "Attach interpreter".

Aha, thank you. Would you be open for a PR which omits the empty arg if the name is empty to help avoid confusion?

[GrahamDumpleton]

I would think it would be better to change how the messages are written.

So if application group is empty use:

Attach main interpreter.
Cleanup main interpreter.

If it is non empty, use:

Attach sub interpreter 'name'.
Cleanup sub interpreter 'name'.

That would make the distinction clearer.

[michael-o]

I would think it would be better to change how the messages are written.

So if application group is empty use:

Attach main interpreter.
Cleanup main interpreter.

If it is non empty, use:

Attach sub interpreter 'name'.
Cleanup sub interpreter 'name'.

That would make the distinction clearer.

ACK, this makes sense. Will provide a PR next week.