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

📄 docs: improve csrf docs #2726

Merged
merged 9 commits into from
Nov 16, 2023
Merged

📄 docs: improve csrf docs #2726

merged 9 commits into from
Nov 16, 2023

Conversation

sixcolors
Copy link
Member

@sixcolors sixcolors commented Nov 15, 2023 •
edited
Loading

Description

  • fix issues with X-Csrf-Token capitalization inconsistency.
  • add package name to ConfigDefault.HandlerContextKey in example.
  • reduce redundancy and repetition.
  • improve grammar.

Type of change

  • Documentation update

@sixcolors
docs: improve csrf docs
e3cfe10 
- fix issues with `X-Csrf-Token` capitalization inconsistency.
- reduce redundancy and repetition.
- improve grammar.
@sixcolors sixcolors requested a review from gaby November 15, 2023 17:50
gaby
gaby approved these changes Nov 16, 2023
View reviewed changes
Copy link
Member

@gaby gaby left a comment

Choose a reason for hiding this comment

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

It's easier to read now. 💪

@ReneWerner87 ReneWerner87 merged commit 2374cad into gofiber:master Nov 16, 2023
20 checks passed
@sixcolors sixcolors deleted the improve-csrf-docs branch November 16, 2023 13:10
@ReneWerner87 ReneWerner87 added this to the Next v2 Release milestone Nov 22, 2023
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

@ReneWerner87 ReneWerner87 ReneWerner87 approved these changes

@efectn efectn Awaiting requested review from efectn

Assignees
No one assigned
Labels
📒 Documentation 🧹 Updates
Projects
None yet
Milestone
v2.52.0
Development

Successfully merging this pull request may close these issues.
3 participants
@sixcolors @gaby @ReneWerner87