From 62976cba1f630bc6560cd2ea5ae2a2fa3082e077 Mon Sep 17 00:00:00 2001
From: Ian Renton <ian@ianrenton.com>
Date: Sun, 27 Sep 2026 14:44:17 +0100
Subject: [PATCH] Enable POTA upstream logging as this was previously proved.
 #95 Improve logging

---
 providers/spot/pota.py            |  3 +--
 templates/add-spot.html           |  2 +-
 templates/alerts.html             |  2 +-
 templates/bands.html              |  4 ++--
 templates/base.html               | 10 +++++-----
 templates/conditions.html         |  2 +-
 templates/map.html                |  4 ++--
 templates/spots.html              |  4 ++--
 templates/status.html             |  2 +-
 webserver/handlers/api/addspot.py |  4 ++++
 10 files changed, 20 insertions(+), 17 deletions(-)

diff --git a/providers/spot/pota.py b/providers/spot/pota.py
index 8c26021..d94bc92 100644
--- a/providers/spot/pota.py
+++ b/providers/spot/pota.py
@@ -57,9 +57,8 @@ class POTA(HTTPSpotProvider):
             new_spots.append(spot)
         return new_spots
 
-    # Upstream spotting disabled until properly tested
     def can_submit_spot(self, activity):
-        return False  # return activity == ActivityName.POTA
+        return activity == ActivityName.POTA
 
     def submit_spot(self, spot, credentials):
         ref_id = spot.activity_refs[0].id if spot.activity_refs else None
diff --git a/templates/add-spot.html b/templates/add-spot.html
index 35b8031..b91e695 100644
--- a/templates/add-spot.html
+++ b/templates/add-spot.html
@@ -130,7 +130,7 @@
     const ALLOW_UPSTREAM_SPOTTING = {% raw safe_json_dumps(web_ui_options["allow_upstream_spotting"]) %};
 </script>
 
-<script src="/static/js/add-spot.js?v=1790516268"></script>
+<script src="/static/js/add-spot.js?v=1790516659"></script>
 <script>$(document).ready(function () {
     $("#nav-link-add-spot").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/alerts.html b/templates/alerts.html
index 8616a2c..6112e3a 100644
--- a/templates/alerts.html
+++ b/templates/alerts.html
@@ -87,7 +87,7 @@
 
 </div>
 
-<script src="/static/js/alerts.js?v=1790516268"></script>
+<script src="/static/js/alerts.js?v=1790516658"></script>
 <script>$(document).ready(function () {
     $("#nav-link-alerts").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/bands.html b/templates/bands.html
index 1a63844..d568fbd 100644
--- a/templates/bands.html
+++ b/templates/bands.html
@@ -82,8 +82,8 @@
     const BANDS = {% raw safe_json_dumps(options["bands"]) %};
 </script>
 
-<script src="/static/js/spotsbandsandmap.js?v=1790516268"></script>
-<script src="/static/js/bands.js?v=1790516268"></script>
+<script src="/static/js/spotsbandsandmap.js?v=1790516658"></script>
+<script src="/static/js/bands.js?v=1790516658"></script>
 <script>$(document).ready(function () {
     $("#nav-link-bands").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/base.html b/templates/base.html
index cc1b96d..d7915f0 100644
--- a/templates/base.html
+++ b/templates/base.html
@@ -1,6 +1,6 @@
 {% extends "skeleton.html" %}
 {% block head_extra %}
-<link rel="stylesheet" href="/static/css/style.css?v=1790516268" type="text/css">
+<link rel="stylesheet" href="/static/css/style.css?v=1790516658" type="text/css">
 <link href="/static/vendor/css/bootstrap-5.3.8.min.css" rel="stylesheet">
 <link href="/static/vendor/css/fontawesome-6.7.2.min.css" rel="stylesheet">
 <link href="/static/vendor/css/solid-6.7.2.min.css" rel="stylesheet">
@@ -16,10 +16,10 @@
     window.fetchEventSource = fetchEventSource;
 </script>
 
-<script src="/static/js/utils.js?v=1790516268"></script>
-<script src="/static/js/ui-ham.js?v=1790516268"></script>
-<script src="/static/js/geo.js?v=1790516268"></script>
-<script src="/static/js/common.js?v=1790516268"></script>
+<script src="/static/js/utils.js?v=1790516658"></script>
+<script src="/static/js/ui-ham.js?v=1790516658"></script>
+<script src="/static/js/geo.js?v=1790516658"></script>
+<script src="/static/js/common.js?v=1790516658"></script>
 {% end %}
 {% block body %}
 <div class="container">
diff --git a/templates/conditions.html b/templates/conditions.html
index a5f2b9d..540f93d 100644
--- a/templates/conditions.html
+++ b/templates/conditions.html
@@ -284,7 +284,7 @@
 </div>
 
 <script src="/static/vendor/js/chart-4.4.9.umd.min.js"></script>
-<script src="/static/js/conditions.js?v=1790516268"></script>
+<script src="/static/js/conditions.js?v=1790516658"></script>
 <script>$(document).ready(function () {
     $("#nav-link-conditions").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/map.html b/templates/map.html
index e1adf27..f865064 100644
--- a/templates/map.html
+++ b/templates/map.html
@@ -115,8 +115,8 @@
     const CARTODB_API_KEY = "{{ web_ui_options.get('cartodb_api_key', '') }}";
 </script>
 
-<script src="/static/js/spotsbandsandmap.js?v=1790516267"></script>
-<script src="/static/js/map.js?v=1790516267"></script>
+<script src="/static/js/spotsbandsandmap.js?v=1790516657"></script>
+<script src="/static/js/map.js?v=1790516657"></script>
 <script>$(document).ready(function () {
     $("#nav-link-map").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/spots.html b/templates/spots.html
index 95ea600..b6959d4 100644
--- a/templates/spots.html
+++ b/templates/spots.html
@@ -127,8 +127,8 @@
 
 </div>
 
-<script src="/static/js/spotsbandsandmap.js?v=1790516267"></script>
-<script src="/static/js/spots.js?v=1790516267"></script>
+<script src="/static/js/spotsbandsandmap.js?v=1790516657"></script>
+<script src="/static/js/spots.js?v=1790516657"></script>
 <script>$(document).ready(function () {
     $("#nav-link-spots").addClass("active");
 }); <!-- highlight active page in nav --></script>
diff --git a/templates/status.html b/templates/status.html
index 005f30d..0ac0e07 100644
--- a/templates/status.html
+++ b/templates/status.html
@@ -96,7 +96,7 @@
     </div>
 </div>
 
-<script src="/static/js/status.js?v=1790516268"></script>
+<script src="/static/js/status.js?v=1790516658"></script>
 <script>
     $(document).ready(function () {
         $("#nav-link-status").addClass("active");
diff --git a/webserver/handlers/api/addspot.py b/webserver/handlers/api/addspot.py
index 1bc1580..1ea2242 100644
--- a/webserver/handlers/api/addspot.py
+++ b/webserver/handlers/api/addspot.py
@@ -214,6 +214,7 @@ class APISpotHandler(tornado.web.RequestHandler):
                 spot.source = "API"
                 spot.infer_missing()
                 self._spots.set(spot.id, spot)
+                logger.info(f"Spot of {spot.dx_call} by {spot.de_call} added to Spothole.")
 
             if upstream_warnings:
                 saved_locally_note = "" if any_upstream_succeeded else " The spot was saved to Spothole only."
@@ -245,6 +246,9 @@ class APISpotHandler(tornado.web.RequestHandler):
             # Submit spot to the upstream provider. Run in a separate thread otherwise this blocks the whole web server
             # for everyone!
             await IOLoop.current().run_in_executor(None, provider.submit_spot, spot, credentials)
+            logger.info(
+                f"Spot of {spot.dx_call} by {spot.de_call} submitted upstream to {upstream_provider_name}."
+            )
             # Trigger a re-poll after 3 second so the spot appears quickly. (Submitting to a cluster node is slower than
             # this, but we get data as a live stream from cluster anyway, so force_poll does nothing in that case. This
             # is really just for the HTTP providers when we submit a spot to them)