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.

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

Add disable html support to monitor middleware. #1620

Merged
merged 5 commits into from Dec 5, 2021

Conversation

efectn
Copy link
Member

@efectn efectn commented Nov 12, 2021

Add disable html support to monitor middleware.

@hi019
Copy link
Member

hi019 commented Nov 12, 2021

I think APIOnly would be more descriptive name

@efectn
Copy link
Member Author

efectn commented Nov 13, 2021

I think APIOnly would be more descriptive name

Yeah, should be better. I'll change variable name and fix tests.

@efectn
Copy link
Member Author

efectn commented Nov 14, 2021

@ReneWerner87 can you help me to fix test?

middleware/monitor/README.md Outdated Show resolved Hide resolved
@ReneWerner87
Copy link
Member

ReneWerner87 commented Nov 17, 2021

sure .... its a very strange bug in the tests

middleware/monitor/config.go Outdated Show resolved Hide resolved
@efectn
Copy link
Member Author

efectn commented Dec 5, 2021

@ReneWerner87 can you check the PR?

@ReneWerner87
Copy link
Member

ReneWerner87 commented Dec 5, 2021

Sure

@ReneWerner87
Copy link
Member

ReneWerner87 commented Dec 5, 2021

Please add this new configuration flag in our documentation repository

@ReneWerner87 ReneWerner87 merged commit b746767 into gofiber:master Dec 5, 2021
14 checks passed
@ReneWerner87
Copy link
Member

ReneWerner87 commented Dec 6, 2021

doc added gofiber/docs@b67c531

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants