Skip to content

Conversation

@MarcelBochtler
Copy link
Member

Add the value of the X-Client-Type header for tracking UI and CLI clients to the MDC logs. This can be used to distinguish all API requests originating from the known clients from unknown ones.

@MarcelBochtler MarcelBochtler force-pushed the log-api-user branch 6 times, most recently from 6962622 to 06477dd Compare December 22, 2025 11:25
@MarcelBochtler MarcelBochtler marked this pull request as ready for review December 22, 2025 11:58
@MarcelBochtler
Copy link
Member Author

The E2E tests are failing because they use the pre-built image from ghcr.
This image does not have the required header change, so every request in the E2E test will fail with a CORS error.

This can be ignored, as the tests will be successful once the image is updated with the change in this PR.

nnobelis
nnobelis previously approved these changes Jan 8, 2026
@MarcelBochtler MarcelBochtler added this pull request to the merge queue Jan 8, 2026
@MarcelBochtler MarcelBochtler removed this pull request from the merge queue due to a manual request Jan 8, 2026
Add the value of the `X-Client-Type` header for tracking UI and CLI
clients to the logs. This can be used to distinguish all API requests
originating from the known clients from unknown ones.

Configure the CLI as `ort-server-cli` and the UI as `ort-server-ui`.
For unknown clients that do not set this header, the value of the
`User-Agent` header will be used.

Signed-off-by: Marcel Bochtler <marcel.bochtler@bosch.com>
@MarcelBochtler MarcelBochtler added this pull request to the merge queue Jan 9, 2026
Merged via the queue into eclipse-apoapsis:main with commit f47b51c Jan 9, 2026
32 of 33 checks passed
@MarcelBochtler MarcelBochtler deleted the log-api-user branch January 9, 2026 18:40
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