-
Notifications
You must be signed in to change notification settings - Fork 159
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DX-1803] /transaction-logs-docs #5932
base: master
Are you sure you want to change the base?
Conversation
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
PR Code Suggestions ✨Explore these optional code suggestions:
|
✅ PS. Pls add /docs/nightly to the end of url
To edit notification comments on pull requests, go to your Netlify site configuration. |
Co-authored-by: Tit Petric <[email protected]>
Co-authored-by: Tit Petric <[email protected]>
Co-authored-by: Tit Petric <[email protected]>
In this PR we are adding a whole new capability to Tyk logging. We can’t just hide it partway down the system log page. These aren’t system logs, but are effectively a duplication of the traffic logs (“analytics”) for a different purpose. So, the documentation for this feature must explain:
We aim to use the principle of Progressive Disclosure in the docs, so I recommend restructuring this page slightly so that it refers to the native logger functionality - the fact it can be configured to write to stderr or stdout (or whatever the options/limitations are), that it recognises different levels of verbosity and how to configure all this. Then we can have a section on System events - and the different messages that are generated - and a separate one on this new access log function. Most of the content is there, but if we want to have useful docs, we need to consider the structure and consistency of the pages, not just add new content to an existing page without checking the flow and usability of that page. |
User description
For internal users - Please add a Jira DX PR ticket to the subject!
DX-1803
https://tyktech.atlassian.net/browse/TT-2539
Preview Link
Description
Screenshots (if appropriate)
Checklist
master
.PR Type
Documentation
Description
Added documentation for enabling API request access logs in Tyk Gateway.
Detailed configuration options for
TYK_ACCESSLOGS_TEMPLATE
explained.Included performance considerations for enabling access logs.
Updated headings format for improved readability.
Changes walkthrough 📝
log-data.md
Document API request access logs and configuration
tyk-docs/content/log-data.md
TYK_ACCESSLOGS_TEMPLATE
.