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

chore: update healcheck dependency #9590

Merged
merged 2 commits into from
Jun 6, 2023

Conversation

AlinsRan
Copy link
Contributor

@AlinsRan AlinsRan commented Jun 2, 2023

Description

This dependency fixes the following issues:

fix: properly log line numbers by using tail calls Kong/lua-resty-healthcheck#29
fix: when not providing a hostname, use IP Kong/lua-resty-healthcheck#48
fix: exit early when reloading during a probe Kong/lua-resty-healthcheck#47
fix: prevent target-list from being nil, due to async behaviour Kong/lua-resty-healthcheck#44
fix: replace timer and node-wide locks with resty-timer, to prevent interval skips Kong/lua-resty-healthcheck#59
fix: do not run out of timers during init/init_worker when adding a vast amount of targets Kong/lua-resty-healthcheck#57
fix: do not call on the module table, but use a method for locks. Also in Kong/lua-resty-healthcheck#57

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@AlinsRan AlinsRan closed this Jun 5, 2023
@AlinsRan AlinsRan reopened this Jun 5, 2023
@AlinsRan AlinsRan marked this pull request as ready for review June 6, 2023 02:13
@monkeyDluffy6017 monkeyDluffy6017 merged commit ec0aa49 into apache:master Jun 6, 2023
42 of 67 checks passed
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.

None yet

3 participants