Skip to content

Conversation

pchickey
Copy link
Contributor

@pchickey pchickey commented Aug 25, 2025

ngx's resolver started as this crate's resolver, and includes bug fixes.

Draft until nginx/ngx-rust#197 lands upstream.

Checklist

Before creating a PR, run through this checklist and mark each as complete:

Copy link

github-actions bot commented Aug 25, 2025

✅ All required contributors have signed the F5 CLA for this PR. Thank you!
Posted by the CLA Assistant Lite bot.

@pchickey pchickey force-pushed the pch/upstreamed_resolver branch from 29d9595 to b6797fd Compare August 25, 2025 16:10
@pchickey
Copy link
Contributor Author

I have hereby read the F5 CLA and agree to its terms

@pchickey pchickey force-pushed the pch/upstreamed_resolver branch from b6797fd to 69c430e Compare September 15, 2025 17:49
@pchickey
Copy link
Contributor Author

Rebased on main and using a patch pointing to the upstreamed nginx/ngx-rust#197 which has now landed. I believe this only needs an ngx-rust release before it can merge.

ngx's resolver started as this crate's resolver, and includes bug fixes
@bavshin-f5 bavshin-f5 force-pushed the pch/upstreamed_resolver branch from 69c430e to 8c5eacc Compare September 15, 2025 23:41
Copy link
Member

@bavshin-f5 bavshin-f5 left a comment

Choose a reason for hiding this comment

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

Thanks a lot!
Updated to ngx v0.5.0, should be good to merge.

@bavshin-f5 bavshin-f5 merged commit 65b35ac into nginx:main Sep 15, 2025
14 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.

2 participants