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

Advanced connectivity policy tests script could not be downloaded #59

Closed
ThilinaManamgoda opened this issue Aug 26, 2021 · 2 comments · Fixed by #61
Closed

Advanced connectivity policy tests script could not be downloaded #59

ThilinaManamgoda opened this issue Aug 26, 2021 · 2 comments · Fixed by #61

Comments

@ThilinaManamgoda
Copy link

When the script is invoked the following error is thrown at the end of the execution,

 Advanced connectivity policy tests script could not be downloaded!
 Confirm this machine can access https://github.com/Azure/SQL-Connectivity-Checker/
 or use a machine with Internet access to see how to run this from machines without Internet. See how at https://github.com/Azure/SQL-Connectivity-Checker/

It seems like the script URL for the Advanced connectivity policy tests script defined with HTTP instead of HTTPS protocol.

Invoke-WebRequest -Uri $('http://raw.githubusercontent.com/Azure/SQL-Connectivity-Checker/' + $RepositoryBranch + '/AdvancedConnectivityPolicyTests.ps1') -OutFile ".\AdvancedConnectivityPolicyTests.ps1" -UseBasicParsing

@ThilinaManamgoda
Copy link
Author

ThilinaManamgoda commented Aug 26, 2021

Also in the configuration snippet given in the README instruction, the following line should should changed to
$scriptUrlBase = 'https://raw.githubusercontent.com/Azure/SQL-Connectivity-Checker/master'
from
$scriptUrlBase = 'http://raw.githubusercontent.com/Azure/SQL-Connectivity-Checker/master

@vitomaz-msft
Copy link
Collaborator

Apologies for the delay getting back to you.
Thank you for the feedback, fixed on v1.35

@vitomaz-msft vitomaz-msft linked a pull request Sep 22, 2021 that will close this issue
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 a pull request may close this issue.

2 participants