Skip to content

Conversation

@liruohrh
Copy link

@liruohrh liruohrh commented Dec 1, 2025

handler.GetLang(internal/base/handler/lang.go) is a err usage? or old usage?
new usage should be GetLangByCtx.
I has check all call, thant i think the right way is call GetLangByCtx but not GetLang,
for middleware of ExtractAndSetAcceptLanguage has registerd.

@LinkinStars LinkinStars self-requested a review December 3, 2025 02:45
@LinkinStars LinkinStars self-assigned this Dec 3, 2025
@LinkinStars LinkinStars changed the base branch from main to dev December 3, 2025 04:03
@LinkinStars
Copy link
Member

Thanks. There are conflicts that need be resolved.

Copy link
Member

@LinkinStars LinkinStars left a comment

Choose a reason for hiding this comment

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

LGTM

@LinkinStars LinkinStars merged commit 740ac61 into apache:dev Dec 8, 2025
@LinkinStars LinkinStars added this to the v1.7.1 milestone Dec 8, 2025
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.

2 participants