Skip to content

{Misc} Fix broken Cloud Shell link #17291

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

Merged
jiasli merged 5 commits into from
Mar 16, 2021
Merged

{Misc} Fix broken Cloud Shell link #17291

Changes from 3 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
246508f
Fix broken Cloud Shell link
Mar 12, 2021
2dd43bc
Link to https://portal.azure.com/
Mar 12, 2021
000fbaa
Use direct link
Mar 12, 2021
60b39f5
Remove /
Mar 12, 2021
4946cea
Merge branch 'dev' into cloud-shell-link
Mar 16, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 1 addition & 3 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@

A great cloud needs great tools; we're excited to introduce *Azure CLI*, our next generation multi-platform command line experience for Azure.

Take a test run now from Azure Cloud Shell!

[![](https://shell.azure.com/images/launchcloudshell.png "Launch Azure Cloud Shell")](https://shell.azure.com)
Take a test run now from [Azure Cloud Shell](https://portal.azure.com/#cloudshell/)!

@jiasli jiasli Mar 12, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://shell.azure.com now redirects to https://portal.azure.com/#cloudshell/

> curl --include https://shell.azure.com
HTTP/1.1 301 Moved Permanently
Content-Length: 160
Content-Type: text/html; charset=UTF-8
Location: https://portal.azure.com/#cloudshell/
Server: Microsoft-IIS/10.0
X-Powered-By: ASP.NET
Date: Fri, 12 Mar 2021 10:40:34 GMT

<head><title>Document Moved</title></head>
<body><h1>Object Moved</h1>This document may be found <a HREF="https://portal.azure.com/#cloudshell/">here</a></body>

So there is no need to use https://shell.azure.com.

The same applies to Azure/azure-powershell#14531.

@jiasli jiasli Mar 12, 2021

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The trailing slash (/) is against the URL syntax:

   The following are two example URIs and their component parts:

         foo://example.com:8042/over/there?name=ferret#nose
         \_/   \______________/\_________/ \_________/ \__/
          |           |            |            |        |
       scheme     authority       path        query   fragment
          |   _____________________|__
         / \ /                        \
         urn:example:animal:ferret:nose

So remove it.


## Installation

Expand Down