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

Do IsInternetActive() less often, and do it more efficiently, fixes #4131 #4134

Merged
merged 1 commit into from
Aug 22, 2022

Conversation

rfay
Copy link
Member

@rfay rfay commented Aug 20, 2022

The Problem/Issue/Bug:

How this PR Solves The Problem:

  • Only do IsInternetActive on start/restart
  • Only test test.ddev.site (and always use that instead of a random url)

Manual Testing Instructions:

export DDEV_DEBUG=true

  • You should only see the check happen on ddev start and ddev restart

Automated Testing Overview:

Related Issue Link(s):

Release/Deployment notes:

@github-actions
Copy link

@rfay rfay merged commit 16ada4c into ddev:master Aug 22, 2022
@rfay rfay deleted the 20220819_improve_internet_detection branch August 22, 2022 00:16
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.

1 participant