-
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
Merged
scottaddie
merged 38 commits into
dotnet:main
from
alexwolfmsft:visual-studio-code-updates
Sep 24, 2025
Merged
Changes from all commits
Commits
Show all changes
38 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 9e6b36b
add vs code flow
alexwolfmsft c91f2ba
add package
alexwolfmsft 306c947
edit pass
alexwolfmsft 12d7c44
fix blank lines
alexwolfmsft e475446
move vs code tab
alexwolfmsft 291ccc7
fix diagram and tabs
alexwolfmsft b589c40
fix image name
alexwolfmsft a38dab8
fix intro
alexwolfmsft d8564e7
fix cli verbiage
alexwolfmsft 0189e51
Apply suggestions from code review
alexwolfmsft cd38539
Apply suggestions from code review
alexwolfmsft 540e6a2
fixes
alexwolfmsft f8e8994
merge
alexwolfmsft f724838
fixes
alexwolfmsft ed31484
fix link
alexwolfmsft 6b79caa
Apply suggestions from code review
alexwolfmsft 08d44fb
Apply suggestions from code review
alexwolfmsft 451187d
refactor intro
alexwolfmsft 1b91c90
fix link
alexwolfmsft 98d72c3
refactor
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
Some comments aren't visible on the classic Files Changed page.
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
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
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.
48 changes: 48 additions & 0 deletions
48
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,48 @@ | ||
--- | ||
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 --> UserPrincipal["**User principal**"] | ||
|
||
%% Azure Branch | ||
Q1 --> AzureApp[Azure] | ||
AzureApp --> ManagedId["**Managed identity**"] | ||
|
||
|
||
%% On-premises Server Branch | ||
Q1 --> OnPremApp[On-premises server] | ||
OnPremApp --> ServicePrincipal["**Service principal**"] | ||
OnPremApp --> ArcManagedId["**Managed identity (Azure Arc only)**"] | ||
|
||
%% 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,UserPrincipal,ManagedId,ServicePrincipal,ArcManagedId authMethod | ||
class LocalDev,AzureApp,OnPremApp envNode | ||
``` |
alexwolfmsft marked this conversation as resolved.
Show resolved
Hide resolved
|
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.