azure devops integration failing about betterscan HOT 5 CLOSED

hi @dkajla

Make sure it is a Git repo, otherwise it will also error out.

On a mobile now. Try this:

  - job: SAST
    displayName: Static Application Security Test (SAST)
    condition: eq(variables['Build.SourceBranch'], 'refs/heads/main')
    pool:
      vmImage: 'ubuntu-latest'
    container: 'scanmycode/scanmycode3-ce:worker-cli'
    steps:
    - script: |
        sudo apt-get update
        sudo apt-get install git-lfs
      displayName: Install git LFS
      
    - checkout: self
      persistCredentials: true

    - script: |
        set -e
        git config --global user.email "[email protected]"
        git config --global user.name "Azure DevOps" 
        git checkout -b $(Build.SourceBranchName)
        sudo CODE_DIR=$(Build.SourcesDirectory) checkmate init
        sudo CODE_DIR=$(Build.SourcesDirectory) checkmate git init
        sudo CODE_DIR=$(Build.SourcesDirectory) checkmate git analyze --branch $(Build.SourceBranchName)
        checkmate issues html
      displayName: Static Application Security Test (SAST)
      env:
        CODE_DIR: '$(Build.SourcesDirectory)'

    - task: PublishBuildArtifacts@1
      displayName: Publish SAST report
      inputs:
        PathtoPublish: $(Build.SourcesDirectory)
        ArtifactName: CodeAnalysisLogs

    - script: |
        git add .checkmate/db.sqlite
        git commit -m '[no ci] update checkmate db'
        git push origin $(Build.SourceBranchName):$(Build.SourceBranch)
      displayName: Commit and Push checkmate db

Notice two new lines:

        sudo CODE_DIR=$(Build.SourcesDirectory) checkmate init
        sudo CODE_DIR=$(Build.SourcesDirectory) checkmate git init

Please let me know how it works.

Also can you scan this repo and send screenshot from Azure DevOps server showing results:

https://github.com/topcodersonline/samplecode

from betterscan.

Hi @marcinguy Thanks for the response.

Yes it did fix the issue I was facing. Additionally, I had to add sudo CODE_DIR=$(Build.SourcesDirectory) ahead of checkmate issues html for the job to complete successfully.

I have few more questions :)

1. Is the final script needed ? which is committing the db file into repo git commit -m '[no ci] update checkmate db, as its triggering pipeline continuously.
2. Is there a option that the tool can scan PR's and add comments automatically. Apologies for multiple questions, I couldn't find any detailed documentation about these configurations.

from betterscan.

hi @devendrakajala

1. this step ensures state is preserved (project will not be rescanned each time for checked files)

I think you can do exclusion for ".checkmate" folder with file types exclusion trigger:

https://stackoverflow.com/questions/55731861/exclude-file-types-in-ci-triggers-on-azure-devops-pipelines-builds

https://learn.microsoft.com/en-us/azure/devops/pipelines/build/triggers?view=azure-devops

2. Scanning PR is possible, then you would see findings for PR only. You should check what Azure DevOps server can do.

Maybe @carlin-q-scott knows more. He contributed Azure DevOps server setup.

P.S @devendrakajala Screenshot of sample repo from Azure DevOps server would be helpful. Maybe you can paste it here or send me on my email. I will add it to documentation.

from betterscan.

@devendrakajala The commit should not be triggering CI, as the [no ci] commit message is supposed to prevent that. I've found that they broke support for that particular message recently, but [ci skip] seems to work for me with a BitBucket repo.

Due to another bug I found with Azure DevOps, I ended up persisting my .checkmate folder to blob storage and leaving it out of the git repo entirely. My issue was that the git credentials aren't persisted properly within a container job when using BitBucket as the git repo.

from betterscan.

Thanks for your comments guys @marcinguy @carlin-q-scott using [ci skip] helped.

from betterscan.