-
Notifications
You must be signed in to change notification settings - Fork 6k
Update visual studio code auth content and diagrams #48182
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
Open
alexwolfmsft
wants to merge
18
commits into
dotnet:main
Choose a base branch
from
alexwolfmsft:visual-studio-code-updates
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+86
−4
Open
Changes from 17 commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
efc7876
update visual studio code auth content
alexwolfmsft bc1052f
fixes
alexwolfmsft 8c4a38f
fix image
alexwolfmsft 03ef5f1
fix diagram
alexwolfmsft aaa4e72
trim image
alexwolfmsft 9bb3d06
images
alexwolfmsft 6e03fa3
Merge branch 'main' into visual-studio-code-updates
alexwolfmsft 41a7f94
standardize
alexwolfmsft 788b37c
fix link
alexwolfmsft 2c0aae8
fixes
alexwolfmsft dce1297
added broker section
alexwolfmsft d7379a8
fix image
alexwolfmsft f72eb22
diagram fixes
alexwolfmsft 02c0579
PR changes
alexwolfmsft 0130b2a
update images
alexwolfmsft 813aeb8
Apply suggestions from code review
alexwolfmsft ad6cdd6
image changes
alexwolfmsft c6e8daf
Feedback changes
alexwolfmsft File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Binary file not shown.
Binary file not shown.
47 changes: 47 additions & 0 deletions
47
docs/azure/sdk/media/mermaidjs/authentication-environments.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,47 @@ | ||
--- | ||
ms.topic: include | ||
ms.date: 08/07/2024 | ||
--- | ||
|
||
```mermaid | ||
%% STEPS TO GENERATE IMAGE | ||
%% ======================= | ||
%% 1. Install mermaid CLI v10.9.1 (see https://github.com/mermaid-js/mermaid-cli/blob/master/README.md): | ||
%% npm i -g @mermaid-js/[email protected] | ||
%% 2. Run command: mmdc -i authentication-environments.md -o ../../media/mermaidjs/authentication-environments.svg | ||
|
||
%%{init: {'theme':'base', 'themeVariables': { 'primaryColor': '#fff', 'edgeLabelBackground':'#fff', 'fontSize': '24px'}}}%% | ||
flowchart LR | ||
NetApp[".NET app"] | ||
Q1{Where is the app running?} | ||
|
||
NetApp --> Q1 | ||
|
||
%% Local Development Machine Branch | ||
Q1 --> LocalDev[Development machine] | ||
LocalDev --> AppSP["**Service principal**"] | ||
LocalDev --> DevAccount["**Developer account**"] | ||
LocalDev --> Broker["**Broker**"] | ||
|
||
%% Azure Branch | ||
Q1 --> AzureApp[Azure] | ||
AzureApp --> ManagedId["**Managed identity**"] | ||
|
||
%% On-premises Server Branch | ||
Q1 --> OnPremApp[On-premises server] | ||
OnPremApp --> ServicePrincipal["**Service principal**"] | ||
|
||
%% Styling | ||
classDef questionBox fill:#4472C4,stroke:#333,stroke-width:2px,color:#fff,font-size:24px | ||
classDef authMethod fill:#e6f2ff,stroke:#4472C4,stroke-width:2px,color:#000,font-size:24px | ||
classDef envNode fill:#8fbc8f,stroke:#333,stroke-width:2px,color:#000,font-size:24px | ||
classDef startNode fill:#2d5f3f,stroke:#333,stroke-width:2px,color:#fff,font-size:24px | ||
|
||
%% Edge label styling | ||
linkStyle default font-size:24px | ||
|
||
class NetApp startNode | ||
class Q1 questionBox | ||
class AppSP,DevAccount,Broker,ManagedId,ServicePrincipal authMethod | ||
class LocalDev,AzureApp,OnPremApp envNode | ||
``` |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
28 changes: 28 additions & 0 deletions
28
docs/azure/sdk/media/mermaidjs/local-broker-authentication.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
--- | ||
ms.topic: include | ||
ms.date: 08/07/2024 | ||
--- | ||
|
||
```mermaid | ||
alexwolfmsft marked this conversation as resolved.
Show resolved
Hide resolved
|
||
%% STEPS TO GENERATE IMAGE | ||
%% ======================= | ||
%% 1. Install mermaid CLI v10.9.1 (see https://github.com/mermaid-js/mermaid-cli/blob/master/README.md): | ||
%% npm i -g @mermaid-js/[email protected] | ||
%% 2. Run command: mmdc -i local-broker-authentication.md -o ../../media/mermaidjs/local-broker-authentication.svg | ||
|
||
flowchart LR | ||
alexwolfmsft marked this conversation as resolved.
Show resolved
Hide resolved
|
||
APP["Local .NET app"] | ||
BK["User credentials supplied by broker"] | ||
AS["Azure services"] | ||
|
||
APP --> BK | ||
BK --> AS | ||
alexwolfmsft marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
classDef app fill:#e6f3ff,stroke:#0078d4,stroke-width:2px,color:#000,font-size:16px | ||
classDef serviceP fill:#D4F4D4,stroke:#7BC97B,stroke-width:2px,color:#000,font-size:16px | ||
classDef services fill:#0078d4,stroke:#005ba1,stroke-width:2px,color:#fff,font-size:16px | ||
|
||
class APP app | ||
class SP serviceP | ||
class AS services | ||
``` |
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
49 changes: 49 additions & 0 deletions
49
docs/azure/sdk/media/mermaidjs/local-developer-authentication.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,49 @@ | ||
--- | ||
ms.topic: include | ||
ms.date: 08/07/2024 | ||
--- | ||
|
||
```mermaid | ||
%% STEPS TO GENERATE IMAGE | ||
%% ======================= | ||
%% 1. Install mermaid CLI v10.9.1 (see https://github.com/mermaid-js/mermaid-cli/blob/master/README.md): | ||
%% npm i -g @mermaid-js/[email protected] | ||
%% 2. Run command: mmdc -i local-developer-authentication.md -o ../../media/mermaidjs/local-developer-authentication.svg | ||
|
||
flowchart TD | ||
ARL[Local .NET app] | ||
|
||
VS[Visual Studio] | ||
VSC[Visual Studio Code] | ||
AZCLI[Azure CLI] | ||
AZPS[Azure PowerShell] | ||
AZD[Azure Developer CLI] | ||
|
||
DevAccount["Developer account credentials"] | ||
|
||
AS["Azure services"] | ||
|
||
ARL --> VS | ||
ARL --> VSC | ||
ARL --> AZD | ||
ARL --> AZCLI | ||
ARL --> AZPS | ||
|
||
VS --> DevAccount | ||
VSC --> DevAccount | ||
AZD --> DevAccount | ||
AZCLI --> DevAccount | ||
alexwolfmsft marked this conversation as resolved.
Show resolved
Hide resolved
|
||
AZPS --> DevAccount | ||
alexwolfmsft marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
DevAccount --> AS | ||
|
||
classDef highlight fill:#0078d4,stroke:#005ba1,stroke-width:2px,color:#fff,font-size:16px | ||
classDef tools fill:#e6f3ff,stroke:#0078d4,stroke-width:1px,font-size:16px | ||
classDef default font-size:16px | ||
classDef lightgreen fill:#D4F4D4,stroke:#7BC97B,stroke-width:2px,color:#000,font-size:16px | ||
|
||
class AS highlight | ||
class VS,VSC,AZD,AZCLI,AZPS tools | ||
class LA,ARL default | ||
class DevAccount lightgreen | ||
``` |
1 change: 1 addition & 0 deletions
1
docs/azure/sdk/media/mermaidjs/local-developer-authentication.svg
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
28 changes: 28 additions & 0 deletions
28
docs/azure/sdk/media/mermaidjs/local-service-principal-authentication.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
--- | ||
ms.topic: include | ||
ms.date: 08/07/2024 | ||
--- | ||
|
||
```mermaid | ||
%% STEPS TO GENERATE IMAGE | ||
%% ======================= | ||
%% 1. Install mermaid CLI v10.9.1 (see https://github.com/mermaid-js/mermaid-cli/blob/master/README.md): | ||
%% npm i -g @mermaid-js/[email protected] | ||
%% 2. Run command: mmdc -i local-service-principal-authentication.md -o ../../media/mermaidjs/local-service-principal-authentication.svg | ||
|
||
flowchart LR | ||
APP["Local .NET app"] | ||
SP["App service principal stored in environment variables"] | ||
AS["Azure services"] | ||
|
||
APP --> SP | ||
SP --> AS | ||
|
||
classDef app fill:#e6f3ff,stroke:#0078d4,stroke-width:2px,color:#000,font-size:16px | ||
classDef serviceP fill:#D4F4D4,stroke:#7BC97B,stroke-width:2px,color:#000,font-size:16px | ||
classDef services fill:#0078d4,stroke:#005ba1,stroke-width:2px,color:#fff,font-size:16px | ||
|
||
class APP app | ||
class SP serviceP | ||
class AS services | ||
``` |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.