From 802e199bf0e022785a934587be6b53673c036bc6 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?D=C3=A1niel=20K=C3=A1ntor?= <git@daniel-kantor.com>
Date: Sat, 8 Jan 2022 08:07:44 +0100
Subject: [PATCH] fix(tools): add integrity checks for the jquery cdn link

---
 apps/tools/librelingo_tools/generate.py                  | 4 +++-
 apps/tools/librelingo_tools/templates/converter.html     | 2 +-
 apps/tools/librelingo_tools/templates/missing_words.html | 2 +-
 apps/tools/practice.py                                   | 9 +++++++--
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/apps/tools/librelingo_tools/generate.py b/apps/tools/librelingo_tools/generate.py
index f3dd5a73055b..84997e20856a 100644
--- a/apps/tools/librelingo_tools/generate.py
+++ b/apps/tools/librelingo_tools/generate.py
@@ -51,7 +51,9 @@ def generate_course(sdir, outdir, tdir, course_dir):
     python = sys.executable
     cmd = f"{python} {lilipy} --course {course_dir} --html {docs_dir}"
     print(cmd)
-    success = os.system(cmd) == 0  # TODO: replace this arbitrary system call with a simple Python call
+    success = (
+        os.system(cmd) == 0
+    )  # TODO: replace this arbitrary system call with a simple Python call
     # {or refactor this file as a shell script. But Python code is preferable)
     os.chdir(current_dir)
     with open(os.path.join(docs_dir, "course.json")) as fh:
diff --git a/apps/tools/librelingo_tools/templates/converter.html b/apps/tools/librelingo_tools/templates/converter.html
index 82974b0c6dc2..70203e2a736e 100644
--- a/apps/tools/librelingo_tools/templates/converter.html
+++ b/apps/tools/librelingo_tools/templates/converter.html
@@ -26,7 +26,7 @@
 <div id="output">
 <div>
 
-<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.6.0/jquery.min.js"></script>
+<script src="https://code.jquery.com/jquery-3.6.0.min.js" integrity="sha256-/xUj+3OJU5yExlq6GSYGSHk7tPXikynS7ogEvDej/m4=" crossorigin="anonymous"></script>
 <script>
 $(document).ready(function(){
     var source_to_target = null;
diff --git a/apps/tools/librelingo_tools/templates/missing_words.html b/apps/tools/librelingo_tools/templates/missing_words.html
index aebd73534bd2..b676f86ee7d1 100644
--- a/apps/tools/librelingo_tools/templates/missing_words.html
+++ b/apps/tools/librelingo_tools/templates/missing_words.html
@@ -23,7 +23,7 @@ <h2>Missing Words</h2>
 </div>
 
 
-<script src="https://ajax.googleapis.com/ajax/libs/jquery/3.6.0/jquery.min.js"></script>
+<script src="https://code.jquery.com/jquery-3.6.0.min.js" integrity="sha256-/xUj+3OJU5yExlq6GSYGSHk7tPXikynS7ogEvDej/m4=" crossorigin="anonymous"></script>
 <script>
 $(document).ready(function(){
     //var source_to_target = null;
diff --git a/apps/tools/practice.py b/apps/tools/practice.py
index 96b1443e8122..e9326cf9028d 100755
--- a/apps/tools/practice.py
+++ b/apps/tools/practice.py
@@ -2,7 +2,12 @@
 import argparse
 import random
 
-from librelingo_tools import guess_path_to_course, collect_data, collect_words, collect_phrases
+from librelingo_tools.lili import (  # type: ignore
+    guess_path_to_course,
+    collect_data,
+    collect_words,
+    collect_phrases,
+)
 from librelingo_yaml_loader.yaml_loader import load_course
 
 
@@ -21,7 +26,7 @@ def get_args():
     return args
 
 
-def check(translation, lookup, word)
+def check(translation, lookup, word):
     return translation.lower() in [wrd.lower() for wrd in lookup[word]]