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

Update "fetch-share-url" to pull correct URL for HTTPS tunnels, not just HTTP #1460

Merged
merged 3 commits into from
Dec 21, 2023

Conversation

rana01645
Copy link
Contributor

Every time I'm using valet fetch-share-url it's giving this error

There is no Ngrok tunnel established for ***********.test

During deep diving, I found that my tunnel is using HTTPS protocol which is why it's unable to get the shared URL!

image

This pull request will solve this issue!

@mattstauffer
Copy link
Collaborator

@rana01645 This is very helpful!

Do you know why you're getting an HTTPS tunnel instead of HTTP, by chance?

@mattstauffer mattstauffer changed the title Give https Protocal support for "valet fetch-share-url" Update "fetch-share-url" to pull correct URL for HTTPS tunnels, not just HTTP Dec 21, 2023
@mattstauffer
Copy link
Collaborator

To be honest, I don't know why we want the HTTP version locally, but I've run into this problem before too, so I'm going to merge it with a hope and a prayer that there's nothing bad about pulling in the HTTPS version when that's all we have.

@mattstauffer mattstauffer merged commit 9c922e5 into laravel:master Dec 21, 2023
4 checks passed
mattstauffer added a commit that referenced this pull request Dec 21, 2023
@rana01645
Copy link
Contributor Author

rana01645 commented Dec 21, 2023

@mattstauffer I also don't know why I'm getting only the HTTPS, we use fetch-share-url to automate the local dev setup using a bash command, and suddenly the automation fails due to this error!

It could be something related to ngrok maybe!

Thanks for merging this pull request.

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