manish1337 commented on PR #72026: URL: https://github.com/apache/airflow/pull/72026#issuecomment-5440122597
Thanks for the review! It only affects `--apps` selections without `core` — core's stack is unchanged. `--apps execution` goes from no access-log middleware to having it; `core` and `all` are untouched. Looks like an oversight, not a decision: all three places that disable the built-in access loggers (`AirflowUvicornWorker.CONFIG_KWARGS`, the uvicorn kwargs in `api_server_command`, `logging_config`) say in their comments that `HttpAccessLogMiddleware` handles access logging — but it was only installed in the `"core" in apps_list` branch. And since the loggers are muted at the logging-config level too, no config can bring the records back. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
