Skip to content

Conversation

@iamajais
Copy link
Contributor

@iamajais iamajais commented Dec 17, 2025

we needed a support for distributed tracing as currently its really hard to debug if the timed-out calls are actually related to a particular scan flow. This interceptor will get the request-id from the request context and set it in the MDC params.

@iamajais iamajais requested a review from a team as a code owner December 17, 2025 07:54
@iamajais iamajais self-assigned this Dec 17, 2025
@iamajais iamajais requested a review from GurtejSohi December 17, 2025 07:54
@github-actions
Copy link

github-actions bot commented Dec 17, 2025

Test Results

90 tests  ±0   90 ✅ ±0   26s ⏱️ -1s
16 suites ±0    0 💤 ±0 
16 files   ±0    0 ❌ ±0 

Results for commit 73f728c. ± Comparison against base commit 8e58f1b.

♻️ This comment has been updated with latest results.

@iamajais iamajais force-pushed the feature/distributed-logging branch from 05b3993 to 67daa75 Compare December 17, 2025 08:00
@iamajais iamajais force-pushed the feature/distributed-logging branch from 67daa75 to 73f728c Compare December 17, 2025 08:03
* needs.
*/
static UUID randomUUID() {
public static UUID randomUUID() {

Choose a reason for hiding this comment

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

Did we change this just for the purpose of the tests

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we needed this implementation of random uuid generation in client side and in our interceptor to generate random uuid as it it faster than traditional UUID generator which requires synchronization between threads.

@iamajais iamajais merged commit 42cb1cf into main Dec 17, 2025
5 checks passed
@iamajais iamajais deleted the feature/distributed-logging branch December 17, 2025 14:18
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