chore: align logging configuration keys under logging.* namespace DHIS2-20960

[teleivo]

Align logging-related dhis.conf keys under a consistent logging.* namespace. These configs are targeted at developers debugging DHIS2 instances, not end users, so we do not need to maintain backward compatibility.

logging.request_id.enabled was renamed to logging.session_id because the old name was misleading. The feature does not deal with request IDs. It adds a hashed session ID to the MDC (sessionId key), which log4j2 can include in log output via %X{sessionId}.

Renames

Old	New
enable.query.logging	logging.query
slow.query.logging.threshold.time	logging.query.slow_threshold
method.query.logging.enabled	logging.query.method
logging.request_id.enabled	logging.session_id

Removed

Key	Reason
elapsed.time.query.logging.enabled	Redundant. It logged Query took Xmsec after every query, which is a noisier, less useful version of what logging.query.slow_threshold already provides (logs slow queries at WARN level with the full query text).

[teleivo]

@netroms is that not a pretty low default? 😅 at least from what I see our queries take

[netroms]

Great! 🙏

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[enricocolasante]

These configs are targeted at developers debugging DHIS2 instances, not end users, so we do not need to maintain backward compatibility.

I agree but I would still find a way to communicate these changes (not sure if it make sense to have it in release notes or have a message on some slack channel) so anyone interested is warned that they have changed. I am just thinking that for @bobjolliffe or anyone like him would be helpful to know that they changed.

[teleivo]

These configs are targeted at developers debugging DHIS2 instances, not end users, so we do not need to maintain backward compatibility.

I agree but I would still find a way to communicate these changes (not sure if it make sense to have it in release notes or have a message on some slack channel) so anyone interested is warned that they have changed. I am just thinking that for @bobjolliffe or anyone like him would be helpful to know that they changed.

I'll write it into the release notes 👌🏻