Skip to content

Conversation

@ofir-frd
Copy link

Benchmark PR langgenius#29015

Type: Corrupted (contains bugs)

Original PR Title: feat: http response add x-trace-id
Original PR Description: > [!IMPORTANT]

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

resolve issue langgenius#29017 add trace_id in http response header and add trace_id in log for debug

for flask request you should enable export ENABLE_REQUEST_LOGGING=true

Screenshots

Before After
image image

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran dev/reformat(backend) and cd web && npx lint-staged(frontend) to appease the lint gods

Original PR URL: langgenius#29015

Compliance Violation

  • Rule: Write Self-Documenting Code With Intent-Explaining Comments
  • Language: Python
  • File: api/extensions/ext_request_logging.py

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