Skip to content

feature: support for adding the specified request header for upstream…#818

Merged
moonming merged 2 commits into
apache:masterfrom
membphis:healthcheck-header
Dec 4, 2019
Merged

feature: support for adding the specified request header for upstream…#818
moonming merged 2 commits into
apache:masterfrom
membphis:healthcheck-header

Conversation

@membphis

@membphis membphis commented Nov 7, 2019

Copy link
Copy Markdown
Member

… health check.

Fix #766.

Comment thread t/admin/health-check.t
Comment thread doc/health-check.md
The threshold fields of `unhealthy` are:
* `active.unhealthy.interval`: Interval between health checks for unhealthy targets (in seconds), the minimum is 1.
* `active.unhealthy.http_failures`: The number of http failures times to determine the target is unhealthy, the minimum is 1.
* `active.req_headers`: Additional request header.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Additional request header. -> Additional request headers. Array format, so you can fill in multiple headers.

@moonming
moonming merged commit 6eca35e into apache:master Dec 4, 2019
@moonming moonming mentioned this pull request Dec 4, 2019
@membphis
membphis deleted the healthcheck-header branch January 7, 2020 02:41
shreemaan-abhishek pushed a commit to shreemaan-abhishek/apisix that referenced this pull request Jan 2, 2026
## Description
Upgrade the actions/cache version, otherwise the CI will fail.
   
## Checklist
- [x] I have explained **WHY** we need this PR and **HOW** the problem
it solves
- [ ] The title of the PR concisely describes the code modification and
complies with APISIX specifications.
 - [ ] I have explained the changes or the new features added to this PR
 - [ ] I have added **e2e test cases** corresponding to this change
 - [ ] I have updated the documentation to reflect this change

**NOTE: This PR will not be reviewed and merged until the contents of
these checklists are completed.**

Signed-off-by: ashing <[email protected]>
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.

feature: support for adding the specified request header for upstream health check.

2 participants