Revert "Docker: Support ARG FROM"

This reverts commit b93ae98.

docker/lib/dependabot/docker/file_parser.rb

@@ -16,16 +16,15 @@ class FileParser < Dependabot::FileParsers::Base
       # Details of Docker regular expressions is at
       # https://github.com/docker/distribution/blob/master/reference/regexp.go
       DOMAIN_COMPONENT =
-        /[[:alnum:]]|[[:alnum:]][[:alnum:]-]*[[:alnum:]]/.freeze
+        /(?:[[:alnum:]]|[[:alnum:]][[[:alnum:]]-]*[[:alnum:]])/.freeze
       DOMAIN = /(?:#{DOMAIN_COMPONENT}(?:\.#{DOMAIN_COMPONENT})+)/.freeze
       REGISTRY = /(?<registry>#{DOMAIN}(?::\d+)?)/.freeze
 
-      NAME_COMPONENT = /[a-z\d]+(?:(?:[._]|__|[-]*)[a-z\d]+)*/.freeze
+      NAME_COMPONENT = /(?:[a-z\d]+(?:(?:[._]|__|[-]*)[a-z\d]+)*)/.freeze
       IMAGE = %r{(?<image>#{NAME_COMPONENT}(?:/#{NAME_COMPONENT})*)}.freeze
 
-      ARG = /ARG/i.freeze
       FROM = /FROM/i.freeze
-      PLATFORM = /--platform=(?<platform>\S+)/.freeze
+      PLATFORM = /--platform\=(?<platform>\S+)/.freeze
       TAG = /:(?<tag>[\w][\w.-]{0,127})/.freeze
       DIGEST = /@(?<digest>[^\s]+)/.freeze
       NAME = /\s+AS\s+(?<name>[\w-]+)/.freeze
@@ -39,14 +38,7 @@ def parse
         dependency_set = DependencySet.new
 
         dockerfiles.each do |dockerfile|
-          args = {}
           dockerfile.content.each_line do |line|
-            if ARG.match(line)
-              key_value = line.delete_prefix("ARG ").split("=")
-              args[key_value[0]] = key_value[1].delete_suffix("\n")
-              next
-            end
-            line = replace_args(line, args)
             next unless FROM_LINE.match?(line)
 
             parsed_from_line = FROM_LINE.match(line).named_captures
@@ -74,13 +66,6 @@ def parse
 
       private
 
-      def replace_args(line, args)
-        line.gsub(/\${?\w+}?/) do |s|
-          escaped = s.delete_prefix("$").delete_prefix("{").delete_suffix("}")
-          args[escaped]
-        end
-      end
-
       def dockerfiles
         # The Docker file fetcher only fetches Dockerfiles, so no need to
         # filter here

docker/spec/dependabot/docker/file_parser_spec.rb

@@ -124,32 +124,6 @@
       end
     end
 
-    context "arg from" do
-      let(:dockerfile_fixture_name) { "arg_from" }
-
-      describe "no curls" do
-        subject(:dependency) { dependencies.first }
-
-        it "can solve the dependency" do
-          expect(dependency).to be_a(Dependabot::Dependency)
-          expect(dependency.name).to eq("docker")
-        end
-      end
-    end
-
-    context "arg from" do
-      let(:dockerfile_fixture_name) { "arg_from_curls" }
-
-      describe "with curls" do
-        subject(:dependency) { dependencies.first }
-
-        it "can solve the dependency" do
-          expect(dependency).to be_a(Dependabot::Dependency)
-          expect(dependency.name).to eq("docker")
-        end
-      end
-    end
-
     context "with a non-numeric version" do
       let(:dockerfile_body) { "FROM ubuntu:artful" }