Skip to content

feat: passed request_id as a header to microservices#23

Merged
TunixR merged 1 commit intomainfrom
feature/21-request-id_header
Nov 18, 2024
Merged

feat: passed request_id as a header to microservices#23
TunixR merged 1 commit intomainfrom
feature/21-request-id_header

Conversation

@TunixR
Copy link
Copy Markdown
Contributor

@TunixR TunixR commented Nov 18, 2024

This pull request includes changes to the src/main.rs file to enhance request handling by adding a new request_id parameter. The changes primarily focus on modifying the build_downstream_request function to include this new parameter and properly forward it as a header.

Enhancements to request handling:

  • src/main.rs: Modified async fn handle_request to pass the request_id parameter to build_downstream_request.
  • src/main.rs: Updated the function signature of async fn build_downstream_request to include the request_id parameter.
  • src/main.rs: Added logic to insert the request_id as an x-request-id header in the build_downstream_request function.

@TunixR TunixR self-assigned this Nov 18, 2024
@TunixR TunixR requested a review from Copilot November 18, 2024 16:50
@TunixR TunixR linked an issue Nov 18, 2024 that may be closed by this pull request
1 task
@TunixR TunixR removed the request for review from adrrf November 18, 2024 16:50
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no suggestions.

Copy link
Copy Markdown
Contributor

@alvarobernal2412 alvarobernal2412 left a comment

Choose a reason for hiding this comment

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

Claud LGTM

@TunixR TunixR merged commit 7836df8 into main Nov 18, 2024
@TunixR TunixR deleted the feature/21-request-id_header branch November 18, 2024 17:00
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.

[FEATURE] Pass request_id to microservices as a header

3 participants