Skip to content

structure the logs - #3188

Merged
JacobCoffee merged 1 commit into
mainfrom
structured-logs
Oct 21, 2026
Merged

JacobCoffee merged 1 commit into
mainfrom
structured-logs

Conversation

@JacobCoffee

Copy link
Copy Markdown
Member

Use django-structlog along with nginx config to structure log output on deploys

Local logs use structlog.dev.ConsoleRenderer so its pretty/easy to ready still when developing

Copilot AI balanced review requested due to automatic review settings October 11, 2026 23:51

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@JacobCoffee
JacobCoffee requested a review from Alc-Alc October 19, 2026 21:43
@JacobCoffee
JacobCoffee marked this pull request as ready for review October 19, 2026 21:43
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-19T21:46:08.834645Z 7caddec Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@Alc-Alc Alc-Alc left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reviewing is hard because logging API is just not the greatest

@JacobCoffee
JacobCoffee added this pull request to the merge queue Oct 21, 2026
Merged via the queue into main with commit e9b5bed Oct 21, 2026
12 checks passed
@JacobCoffee
JacobCoffee deleted the structured-logs branch October 21, 2026 13:52
@jrobichaud

Copy link
Copy Markdown

Hi, I am the author of django-structlog.

I noticed you started using django-structlog and you also use celery.

I'd like to make sure you are aware that django-structlog also offers a celery integration.

When configured, it automatically logs celery events (ex task_started, task_finished, etc.) and it also binds useful metadata to the logs (task_id, the request_id of the request that initated the task, parent_it, and more).

I just wanted to be sure you did not miss this because I believe this could be useful for you.

Have a great day.

https://django-structlog.readthedocs.io/en/latest/events.html#celery
https://django-structlog.readthedocs.io/en/latest/celery.html#getting-started-with-celery

@JacobCoffee

Copy link
Copy Markdown
Member Author

thanks @jrobichaud ! we will do a follow up to add structure any missed logs. @Alc-Alc want to add this to list?

@Alc-Alc

Alc-Alc commented Oct 25, 2026 •

Copy link
Copy Markdown

@Alc-Alc want to add this to list?

sure, I will have it done later today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants