Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

fix(fiberi18n): use sync.Map and utils.CopyString to fix concurrent problem #718

Merged
merged 1 commit into from
Aug 22, 2023
Merged

fix(fiberi18n): use sync.Map and utils.CopyString to fix concurrent problem #718

merged 1 commit into from
Aug 22, 2023

Conversation

Skyenought
Copy link
Member

@Skyenought Skyenought commented Aug 17, 2023 •
edited
Loading

fix #594

@github-actions github-actions bot added the ☢️ Bug Something isn't working label Aug 17, 2023
@Skyenought Skyenought force-pushed the i18n/bug branch 2 times, most recently from 8331e79 to f268136 Compare August 17, 2023 19:40
@gaby gaby requested a review from efectn August 19, 2023 20:16
@Skyenought Skyenought force-pushed the i18n/bug branch 2 times, most recently from e4ecacb to 5d1eac5 Compare August 22, 2023 17:59
@Skyenought Skyenought merged commit aefb4dc into gofiber:main Aug 22, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gaby gaby gaby approved these changes

@efectn efectn efectn approved these changes

@ReneWerner87 ReneWerner87 Awaiting requested review from ReneWerner87

Assignees
No one assigned
Labels
☢️ Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

fiberi18n error on concurrent requests
3 participants
@Skyenought @gaby @efectn