Skip to content

feat: implement application logs storage#53

Draft
TranTran365 wants to merge 2 commits intomainfrom
feat/logging
Draft

feat: implement application logs storage#53
TranTran365 wants to merge 2 commits intomainfrom
feat/logging

Conversation

@TranTran365
Copy link
Copy Markdown

middleware.ts captures incoming requests and sends logs to route.ts. The route then calls logRequest() to store the data in Appwrite.

@TranTran365 TranTran365 requested a review from thu4n March 13, 2026 14:39
@snyk-io
Copy link
Copy Markdown

snyk-io Bot commented Mar 13, 2026

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@thu4n thu4n marked this pull request as draft March 22, 2026 02:06
Comment thread middleware.ts
Comment thread middleware.ts
Comment on lines +24 to +30
const log = {
ip: req.headers.get("x-forwarded-for") || req.headers.get("x-real-ip") || "unknown",
path,
method,
userAgent,
status: response.status,
}
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.

Need more info than this, see current quamon site logs for reference

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.

3 participants