Skip to content

feat(ops): check dns resolver address is valid#7079

Merged
spacewander merged 10 commits into
apache:masterfrom
tzssangglass:dns_resolver
May 25, 2022
Merged

feat(ops): check dns resolver address is valid#7079
spacewander merged 10 commits into
apache:masterfrom
tzssangglass:dns_resolver

Conversation

@tzssangglass

Copy link
Copy Markdown
Member

Description

Fixes # (issue)

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)

Comment thread apisix/cli/ops.lua Outdated
@tzssangglass
tzssangglass requested a review from spacewander May 21, 2022 03:04
Comment thread apisix/cli/ops.lua Outdated
Comment thread t/cli/test_dns.sh
spacewander
spacewander previously approved these changes May 23, 2022
Comment thread apisix/cli/ops.lua Outdated
soulbird
soulbird previously approved these changes May 25, 2022
@tzssangglass
tzssangglass dismissed stale reviews from soulbird and spacewander via 4813ea7 May 25, 2022 01:52
@spacewander
spacewander merged commit d6f8b2a into apache:master May 25, 2022
hongbinhsu pushed a commit to fitphp/apix that referenced this pull request May 25, 2022
* upstream/master:
  test: make sure the nginx.pid is written (apache#7122)
  fix(grpc-transcode): pb_option should be set on the route (apache#7116)
  feat(ops):   check dns resolver address is valid (apache#7079)
  fix: reduce memory usage when abnormal weights are given in chash (apache#7103)
@tzssangglass
tzssangglass deleted the dns_resolver branch October 10, 2022 10:42
Liu-Junlin pushed a commit to Liu-Junlin/apisix that referenced this pull request Nov 4, 2022
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.

4 participants