Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

Add option to disable writing headers - #19

Merged
mitchellh merged 1 commit into
mitchellh:masterfrom
cljoly:disable-headers
Nov 8, 2020
Merged

Add option to disable writing headers#19
mitchellh merged 1 commit into
mitchellh:masterfrom
cljoly:disable-headers

Conversation

@cljoly

@cljoly cljoly commented Oct 12, 2020

Copy link
Copy Markdown
Contributor

This option prevents metrics from being written in the corresponding
header of the response.

This is useful to collect the metrics all the time but have a setting to
actually write it in the answer. For instance, this is desirable for
miniflux/v2#806

This option prevents metrics from being written in the corresponding
header of the response.

This is useful to collect the metrics all the time but have a setting to
actually write it in the answer. For instance, this is desirable for
miniflux/v2#806
@cljoly

cljoly commented Nov 7, 2020

Copy link
Copy Markdown
Contributor Author

@mitchellh ping (in case the notification got lost) :-)

@mitchellh mitchellh left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Looks great. Thanks! Sorry I did lose this in my inbox.

@mitchellh
mitchellh merged commit feb680a into mitchellh:master Nov 8, 2020
@cljoly

cljoly commented Nov 8, 2020

Copy link
Copy Markdown
Contributor Author

Awesome, thanks!

@cljoly
cljoly deleted the disable-headers branch November 8, 2020 09:24
@cljoly

cljoly commented Nov 10, 2020

Copy link
Copy Markdown
Contributor Author

@mitchellh could you please consider tagging a commit with this functionality to ease use in other projects?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants