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.

Sign up for GitHub

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

Jump to bottom

Default to using DNS cache (with max 256 entries) for Connector #226

Merged
merged 1 commit into from
Mar 11, 2020
Merged

Default to using DNS cache (with max 256 entries) for Connector #226

merged 1 commit into from
Mar 11, 2020

Conversation

clue
Copy link
Member

@clue clue commented Mar 11, 2020 •
edited
Loading

Resolves / closes #115
Builds on top of reactphp/dns#127 and reactphp/cache#26

@clue clue added this to the v1.4.0 milestone Mar 11, 2020
@clue clue added the easy pick label Mar 11, 2020
@clue clue requested review from WyriHaximus and jsor March 11, 2020 16:05
@jsor jsor merged commit 550c336 into reactphp:master Mar 11, 2020
@clue clue deleted the caching-dns branch March 11, 2020 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jsor jsor jsor approved these changes

@WyriHaximus WyriHaximus WyriHaximus approved these changes

Assignees
No one assigned
Labels
easy pick new feature
Projects
None yet
Milestone
v1.4.0
Development

Successfully merging this pull request may close these issues.

[Client] Connector should use a caching DNS resolver by default
3 participants
@clue @jsor @WyriHaximus