Skip to content
This repository was archived by the owner on Apr 20, 2023. It is now read-only.

Security build definition for CLI#7076

Closed
ravimeda wants to merge 267 commits intodotnet:masterfrom
ravimeda:sec_build
Closed

Security build definition for CLI#7076
ravimeda wants to merge 267 commits intodotnet:masterfrom
ravimeda:sec_build

Conversation

@ravimeda
Copy link
Copy Markdown
Member

@ravimeda ravimeda commented Jul 5, 2017

Similar to dotnet/core-setup#2722
The approach for acquiring official build blobs is different since CLI does not have a dependency dotnet/buildtools that offers reliable means to download Azure blobs.

Approach:

  1. Clone core-setup, and run init-tools command to acquire BuildTools.
  2. Download blobs with from dotnetcli/Sdk/<branch> . For Windows build, include only blobs with extension .zip or .exe.
  3. Run BinSkim and APIScan on all assemblies and executables in the downloaded blobs.
  4. Get sources corresponding to the official build. This means checkout at the SHA listed latest.version
  5. Run CredScan and PoliCheck on the sources.

@morganbr list of issues found -
https://msazure.visualstudio.com/DefaultCollection/One/_workitems?tempQueryId=740da6e8-15ce-4912-b983-2cf87ed6a530

@livarcocc PTAL.

Post PR, I will update documentation for launching security build for CLI at https://github.com/dotnet/core-eng/blob/master/Documentation/Project-Docs/security-builds.md

https://github.com/dotnet/core-eng/issues/650

KevinRansom and others added 19 commits July 1, 2017 13:07
Revert "Pinning the SDK version to: "2.0.0-preview3-006628""
…upCrossgen229Coherence26027

Update to SetupCrossgen 229 - Coherence 26027
…-to-release/2.0.0-20170703-070029

* dotnet/rel/1.1.0:
  feedback
  Enable VB for dotnet cli 1.1
  MSBuild 15.3.402
  Add F# to dotnet cli 1..1.0
…/UpdateTemplateEngineFor1x

Mlorbetske dev/mlorbe/update template engine for1x
…e/2.0.0-20170703-070029

Merge rel/1.1.0 to release/2.0.0
Update SDK to 2.0.0-preview3-20170703-2
Update runtime to 2.0.0-preview3-25502-01
@ravimeda ravimeda self-assigned this Jul 5, 2017
@ravimeda ravimeda requested review from livarcocc and morganbr July 5, 2017 16:06
livarcocc and others added 7 commits July 5, 2017 09:54
… 1.0.4 if the minimum SDK version file is not found.
…-to-release/2.0.0-20170705-070026

* dotnet/rel/1.1.0:
  Copy Paste error
  chmod .sh files
  Update the SDK to 1.1.0-alpha-20170630-2
  Update Template Engine and templates for 1.x
…e/2.0.0-20170705-070026

Merge rel/1.1.0 to release/2.0.0
…upCrossgen246Coherence26065

Update to SetupCrossgen 249 - Coherence 26071
…ster-20170705-070028

Merge release/2.0.0 to master
github-john-doe and others added 25 commits July 25, 2017 08:27
…ster-20170725-070023

Merge release/2.0.0 to master
Updating the build definition URLs: master
Insert NuGet Build 4.3.0-rtm-4324 into cli
Use 2.1 builds for stage0 of 2.1 builds
…6452

updating to coherence 26452 & templating build 300
…e/2.0.0-20170725-070022

Merge rel/1.1.0 to release/2.0.0
…imestampPackages

Switch to picking up the no-timestamp runtime store and package cache
This change disables security build task that remove non-CLI assemblies. Thereby, all assemblies are scanned, and issues logged through TSA. This means issues in assemblies not owned or produced by CLI, have to be resolved as Won't Fix so that TSA will not re-create these issues in future scans.
Passwords/keys should not be passed in the environment
…ster-20170726-070023

Merge release/2.0.0 to master
This change disables security build task that remove non-CLI assemblies. Thereby, all assemblies are scanned, and issues logged through TSA. This means issues in assemblies not owned or produced by CLI, have to be resolved as Won't Fix so that TSA will not re-create these issues in future scans.
@ravimeda
Copy link
Copy Markdown
Member Author

Please ignore the noise. I'm trying to rebase and resolve merge conflicts. If it does not work, I'll send a fresh PR.

@ravimeda
Copy link
Copy Markdown
Member Author

Please use #7296

@ravimeda ravimeda closed this Jul 27, 2017
@ravimeda ravimeda deleted the sec_build branch November 16, 2017 22:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.