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.

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

Add billing subscription prerequisite and other fixes #694

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add billing subscription prerequisite and other fixes #694

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
993c430
Add billing subscription prerequisite
ReneSchumacher Dec 15, 2023
2d2dc47
Remove tip about squash merge from vulnerability scanning lab
ReneSchumacher Dec 15, 2023
46f923d
Fix ProTip format
ReneSchumacher Dec 15, 2023
3bfae1f
Remove pro tip from code scanning
ReneSchumacher Dec 15, 2023
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
8 changes: 1 addition & 7 deletions labs/azuredevops/advancedsecurity/readme.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ In this lab, you will see how you can use Advanced Security to protect the Azure
### Before you begin

- This lab requires you to complete task 1 from the <a href="../prereq-advsec/">prerequisite</a> instructions.
- Since GitHub Advanced Security for Azure DevOps requires billing, you must have <a href="https://learn.microsoft.com/en-us/azure/devops/organizations/billing/set-up-billing-for-your-organization-vs?view=azure-devops">set up a billing subscription</a> for your organization.


### Task 1: Enable Advanced Security from Portal
Expand Down Expand Up @@ -186,9 +187,6 @@ When a Dependency Alert is created in Azure DevOps Advanced Security, it will co
The build will run automatically, initiating the dependency scanning task and publishing the results to Advanced Security and alert automatically closed.


>**ProTip!** Squash Merge is important. If we just commit, the exposed credential will still be in the history. To avoid this, fix code, use a Squash Merge, push it to repo, and you're done!


1. Once the pipeline has been completed, **eShopOnWeb**, go to the Azure DevOps Advanced Security dashboard and click on Dependencies.

1. You will see that the alert *Improper Input Validation in IpMatcher....*... no longer exists, as it is now closed.
Expand Down Expand Up @@ -371,10 +369,6 @@ Code scanning in GitHub Advanced Security for Azure DevOps lets you analyze the
4. Click on Detections to see the different builds that detected this alert.


> ProTip!
> When a vulnerable component is no longer detected in the latest build for pipelines with the dependency scanning task, the state of the associated alert is automatically changed to Closed. To see these resolved alerts, you can use the State filter in the main toolbar and select Closed.


#### Fixing the Code to resolve the alert
1. This is simple to fix using the method using parameters with dynamic SQL described in the Remediation steps.

Expand Down