Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Mutualise a part of Context between net/http and gin #288

Merged
merged 3 commits into from
Dec 20, 2024

Conversation

EwenQuim
Copy link
Member

@EwenQuim EwenQuim commented Dec 19, 2024

Tried to mutualize the Context so we will have less work to do for other integrations.

Base automatically changed from refactored-context to main December 19, 2024 22:17
@EwenQuim EwenQuim force-pushed the share-context-gin-nethttp branch from f7cd254 to a9b6a2f Compare December 19, 2024 22:24
@EwenQuim EwenQuim changed the title Mutualise some calls between net/http and gin Mutualise a part of Context between net/http and gin Dec 20, 2024
@EwenQuim EwenQuim marked this pull request as ready for review December 20, 2024 17:36
@EwenQuim EwenQuim requested a review from dylanhitt December 20, 2024 17:36
@EwenQuim EwenQuim mentioned this pull request Dec 20, 2024
ctx.go Show resolved Hide resolved
internal/common_context.go Show resolved Hide resolved
@EwenQuim EwenQuim merged commit 2991388 into main Dec 20, 2024
7 checks passed
@EwenQuim EwenQuim deleted the share-context-gin-nethttp branch December 20, 2024 18:22
@EwenQuim EwenQuim mentioned this pull request Dec 22, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants