Skip to content

Conversation

@drbyte
Copy link
Contributor

@drbyte drbyte commented Nov 30, 2019

Dnsmasq allows use of a common config folder, much like nginx and php do.

This PR changes Valet's default config to empower this feature.

This will make for easier troubleshooting
... and easier customizing (such as dropping in a custom logging config, additional TLDs, alternate DNS resolvers, etc)

@drbyte drbyte force-pushed the dnsmasq-customconfig branch 3 times, most recently from e325b35 to 050c7c7 Compare December 1, 2019 05:24
… do.

This PR changes Valet's default config process to empower this feature, which makes installation less intrusive, and easier to identify and remove valet-specific customizations.

This will make for easier troubleshooting
... and easier customizing (such as dropping in a custom logging config, additional TLDs, alternate DNS resolvers, etc)

Also removes old dnsmasq configs used by prior Valet versions
@drbyte drbyte force-pushed the dnsmasq-customconfig branch from 050c7c7 to 02ad7f4 Compare December 2, 2019 20:03
@drbyte
Copy link
Contributor Author

drbyte commented Dec 2, 2019

Updated to include doing a cleanup of the old config file from previous Valet installs.

@mattstauffer mattstauffer merged commit d301e4a into laravel:master Dec 5, 2019
@drbyte drbyte deleted the dnsmasq-customconfig branch December 5, 2019 20:46
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