When we shadow, trust the filenames and file sizes, but treat them as import-error

Fixes #3005

Author: nickygerritsen

webapp/src/Service/SubmissionService.php

@@ -569,7 +569,7 @@ public function submitSolution(
 
         if (!empty($entryPoint) && !preg_match(self::FILENAME_REGEX, $entryPoint)) {
             $message = sprintf("Entry point '%s' contains illegal characters.", $entryPoint);
-            if ($forceImportInvalid) {
+            if ($forceImportInvalid || $source === 'shadowing') {
                 $importError = $message;
             } else {
                 return null;
@@ -610,7 +610,7 @@ public function submitSolution(
             }
             if (!preg_match(self::FILENAME_REGEX, $file->getClientOriginalName())) {
                 $message = sprintf("Illegal filename '%s'.", $file->getClientOriginalName());
-                if ($forceImportInvalid) {
+                if ($forceImportInvalid || $source === 'shadowing') {
                     $importError = $message;
                 } else {
                     return null;
@@ -647,7 +647,7 @@ public function submitSolution(
 
         if ($totalSize > $sourceSize * 1024) {
             $message = sprintf("Submission file(s) are larger than %d kB.", $sourceSize);
-            if ($forceImportInvalid) {
+            if ($forceImportInvalid || $source === 'shadowing') {
                 $importError = $message;
             } else {
                 return null;