Merge pull request #3446 from snyk/fix/bump-docker-plugin-go-modules

fix: bump snyk docker plugin version golang fixes

package.json

@@ -111,7 +111,7 @@
     "semver": "^6.0.0",
     "snyk-config": "4.0.0",
     "snyk-cpp-plugin": "2.20.0",
-    "snyk-docker-plugin": "^4.38.0",
+    "snyk-docker-plugin": "^5.0.1",
     "snyk-go-plugin": "1.19.0",
     "snyk-gradle-plugin": "3.20.2",
     "snyk-module": "3.1.0",

src/cli/commands/monitor/index.ts

@@ -88,6 +88,11 @@ export default async function monitor(...args0: MethodArgs): Promise<any> {
     throw new Error('`--remote-repo-url` is not supported for container scans');
   }
 
+  // TODO remove once https://github.com/snyk/cli/pull/3433 is merged
+  if (options.docker && !options['app-vulns']) {
+    options['exclude-app-vulns'] = true;
+  }
+
   // Handles no image arg provided to the container command until
   // a validation interface is implemented in the docker plugin.
   if (options.docker && paths.length === 0) {

src/cli/commands/test/index.ts

@@ -91,6 +91,11 @@ export default async function test(
     throw new MissingArgError();
   }
 
+  // TODO remove once https://github.com/snyk/cli/pull/3433 is merged
+  if (options.docker && !options['app-vulns']) {
+    options['exclude-app-vulns'] = true;
+  }
+
   const ecosystem = getEcosystemForTest(options);
   if (ecosystem) {
     try {

test/tap/cli-monitor.acceptance.test.ts

@@ -1741,6 +1741,7 @@ if (!isWindows) {
       [
         {
           docker: true,
+          'exclude-app-vulns': true,
           org: 'explicit-org',
           path: 'foo:latest',
         },
@@ -1809,6 +1810,7 @@ if (!isWindows) {
       [
         {
           docker: true,
+          'exclude-app-vulns': true,
           file: 'Dockerfile',
           org: 'explicit-org',
           path: 'foo:latest',
@@ -1876,6 +1878,7 @@ if (!isWindows) {
       [
         {
           docker: true,
+          'exclude-app-vulns': true,
           org: 'explicit-org',
           'policy-path': 'custom-location',
           path: 'foo:latest',
@@ -1939,6 +1942,7 @@ if (!isWindows) {
       [
         {
           docker: true,
+          'exclude-app-vulns': true,
           path: 'foo:latest',
           platform,
         },

test/tap/cli-test/cli-test.docker.spec.ts

@@ -65,6 +65,7 @@ export const DockerTests: AcceptanceTests = {
         [
           {
             docker: true,
+            'exclude-app-vulns': true,
             org: 'explicit-org',
             projectName: null,
             packageManager: null,
@@ -141,6 +142,7 @@ export const DockerTests: AcceptanceTests = {
         [
           {
             docker: true,
+            'exclude-app-vulns': true,
             org: 'explicit-org',
             projectName: null,
             packageManager: null,
@@ -293,6 +295,7 @@ export const DockerTests: AcceptanceTests = {
           {
             file: 'Dockerfile',
             docker: true,
+            'exclude-app-vulns': true,
             org: 'explicit-org',
             projectName: null,
             packageManager: null,
@@ -407,6 +410,7 @@ export const DockerTests: AcceptanceTests = {
         [
           {
             docker: true,
+            'exclude-app-vulns': true,
             org: 'explicit-org',
             projectName: null,
             packageManager: null,
@@ -482,6 +486,7 @@ export const DockerTests: AcceptanceTests = {
         [
           {
             docker: true,
+            'exclude-app-vulns': true,
             org: 'explicit-org',
             projectName: null,
             packageManager: null,
@@ -563,6 +568,7 @@ export const DockerTests: AcceptanceTests = {
         [
           {
             docker: true,
+            'exclude-app-vulns': true,
             org: 'explicit-org',
             projectName: null,
             packageManager: null,