Remove the WeBWorK SOAP service.

I don't think that anyone uses this anymore, and no one should use it.
It is not secure, and there are better ways to interact with webwork,
such as LTI authentication.

Author: drgrice1

Dockerfile

@@ -140,8 +140,6 @@ RUN apt-get update \
 	libpath-tiny-perl \
 	libperl-critic-perl \
 	libphp-serialization-perl \
-	libpod-wsdl-perl \
-	libsoap-lite-perl \
 	libsql-abstract-perl \
 	libssl-dev \
 	libstring-shellquote-perl \
@@ -233,18 +231,18 @@ COPY --from=base /opt/base/pg $APP_ROOT/pg
 RUN echo "PATH=$PATH:$APP_ROOT/webwork2/bin" >> /root/.bashrc \
 	&& mkdir /run/webwork2 /etc/ssl/local \
 	&& cd $APP_ROOT/webwork2/ \
-		&& chown www-data DATA ../courses logs tmp /etc/ssl/local /run/webwork2 \
-		&& chmod -R u+w DATA ../courses logs tmp /run/webwork2 /etc/ssl/local \
+	&& chown www-data DATA ../courses logs tmp /etc/ssl/local /run/webwork2 \
+	&& chmod -R u+w DATA ../courses logs tmp /run/webwork2 /etc/ssl/local \
 	&& echo "en_US ISO-8859-1\nen_US.UTF-8 UTF-8" > /etc/locale.gen \
-		&& /usr/sbin/locale-gen \
-		&& echo "locales locales/default_environment_locale select en_US.UTF-8\ndebconf debconf/frontend select Noninteractive" > /tmp/preseed.txt \
-		&& debconf-set-selections /tmp/preseed.txt \
+	&& /usr/sbin/locale-gen \
+	&& echo "locales locales/default_environment_locale select en_US.UTF-8\ndebconf debconf/frontend select Noninteractive" > /tmp/preseed.txt \
+	&& debconf-set-selections /tmp/preseed.txt \
 	&& rm -f /etc/localtime /etc/timezone && echo "Etc/UTC" > /etc/timezone \
-		&& dpkg-reconfigure -f noninteractive tzdata \
+	&& dpkg-reconfigure -f noninteractive tzdata \
 	&& cd $WEBWORK_ROOT/htdocs \
-		&& npm install \
+	&& npm install \
 	&& cd $PG_ROOT/htdocs \
-		&& npm install
+	&& npm install
 
 # ==================================================================
 # Phase 7 - Final setup and prepare docker-entrypoint.sh

DockerfileStage1

@@ -101,8 +101,6 @@ RUN apt-get update \
 	libpath-tiny-perl \
 	libperl-critic-perl \
 	libphp-serialization-perl \
-	libpod-wsdl-perl \
-	libsoap-lite-perl \
 	libsql-abstract-perl \
 	libssl-dev \
 	libstring-shellquote-perl \

bin/check_modules.pl

@@ -427,17 +427,6 @@ =head1 DESCRIPTION
 			rhel   => 'perl-Pod-Usage'
 		}
 	},
-	'Pod::WSDL' => {
-		package => {
-			ubuntu => 'libpod-wsdl-perl'
-		}
-	},
-	'SOAP::Lite' => {
-		package => {
-			ubuntu => 'libsoap-lite-perl',
-			rhel   => 'perl-SOAP-Lite'
-		}
-	},
 	'SQL::Abstract' => {
 		minversion => '2',
 		package    => {

conf/webwork2.mojolicious.dist.yml

@@ -38,11 +38,6 @@ JSON_ERROR_LOG: 0
 # does not forward the server root location.
 #server_root_url_redirect: https://mysite.edu/mypage.html
 
-# Uncomment the line below to use the WebworkSOAP handlers.  Make sure that you
-# change the value to something more secure.  Note that it must consist entirely
-# of digits.
-#soap_authen_key: 123456789123456789
-
 # If a remote site is using this WW server to process WW problems and embed them
 # into web pages, the web browser might not permit the WW problems to be
 # displayed, as it violates security policy to mix content from different

lib/Mojolicious/WeBWorK.pm

@@ -228,23 +228,6 @@ sub startup ($app) {
 		}
 	);
 
-	if ($config->{soap_authen_key}) {
-		# Only allow an authen key that consists entirely of digits.  The WebworkSOAP module uses a numeric != for
-		# comparison, and in perl all strings containing alphabetic characters are numerically equal.  So if this is not
-		# numeric all keys that are passed in will succeed in authentication.  Very dangerous!
-		if ($config->{soap_authen_key} =~ /^\d*$/) {
-			$app->log->info('SOAP endpoints enabled');
-			$WeBWorK::SeedCE{soap_authen_key} = $config->{soap_authen_key};
-
-			push(@{ $r->namespaces }, 'WebworkSOAP');
-			$r->any('/webwork2_wsdl')->to('SOAP#wsdl');
-			$r->post('/webwork2_rpc')->to('SOAP#dispatch');
-		} else {
-			$app->log->info(qq{Invalid soap_authen_key "$config->{soap_authen_key}". }
-					. 'It must consist entirely of digits.  SOAP endpoints NOT enabled.');
-		}
-	}
-
 	# Letsencrypt renewal route.
 	if ($config->{enable_certbot_webroot_routes}) {
 		$r->any(