fix: Calculate balance with expired domains

This fixes mobile views from splitting the text and icons for the links

account.py

@@ -45,9 +45,7 @@ if HSD_INTERNAL_NODE:
         HSD_API = "firewallet-" + str(int(time.time()))
     HSD_IP = "localhost"
 
-SHOW_EXPIRED = os.getenv("SHOW_EXPIRED")
-if SHOW_EXPIRED is None:
-    SHOW_EXPIRED = False
+SHOW_EXPIRED = os.getenv("SHOW_EXPIRED","false").lower() in ["1","true","yes"]
 
 HSD_PROCESS = None
 SPV_MODE = None
@@ -95,6 +93,7 @@ def hsdConnected():
 def hsdVersion(format=True):
     info = hsd.getInfo()
     if 'error' in info:
+        logger.error(f"HSD connection error: {info.get('error', 'Unknown error')}")
         return -1
     
     # Check if SPV mode is enabled
@@ -121,6 +120,7 @@ def check_account(cookie: str | None):
     # Check if the account is valid
     info = hsw.getAccountInfo(account, 'default')
     if 'error' in info:
+        logger.error(f"HSW error checking account {account}: {info.get('error', 'Unknown error')}")
         return False
     return account
 
@@ -364,7 +364,7 @@ def getBalance(account: str):
     available = available / 1000000
     logger.debug(f"Initial balance for account {account}: total={total}, available={available}, locked={locked}")
 
-    domains = getDomains(account)
+    domains = getDomains(account,True,True)
     domainValue = 0
     domains_to_update = []  # Track domains that need cache updates
     
@@ -426,7 +426,7 @@ def getBalance(account: str):
             daemon=True
         )
         thread.start()
-    
+        
     total = total - (domainValue/1000000)
     locked = locked - (domainValue/1000000)
     logger.debug(f"Adjusted balance for account {account}: total={total}, available={available}, locked={locked}")
@@ -471,14 +471,14 @@ def getPendingTX(account: str):
     return pending
 
 
-def getDomains(account, own=True):
+def getDomains(account, own: bool = True, expired: bool = SHOW_EXPIRED):
     if own:
         response = requests.get(get_wallet_api_url(f"/wallet/{account}/name?own=true"))
     else:
         response = requests.get(get_wallet_api_url(f"/wallet/{account}/name"))
     info = response.json()
 
-    if SHOW_EXPIRED:
+    if expired:
         return info
 
     # Remove any expired domains
@@ -583,6 +583,7 @@ def getTransactions(account, page=1, limit=100):
             return []
         info = hsw.getWalletTxHistory(account)
         if 'error' in info:
+            logger.error(f"Error getting transactions for account {account}: {info['error']}")
             return []
         return info[::-1]
 
@@ -916,6 +917,7 @@ def register(account, domain):
 def getNodeSync():
     response = hsd.getInfo()
     if 'error' in response:
+        logger.error(f"Error getting node sync status: {response['error']}")
         return 0
 
     sync = response['chain']['progress']*100
@@ -1941,13 +1943,17 @@ def hsdStart():
             cmd.append(flag)
 
     # Launch process
-    HSD_PROCESS = subprocess.Popen(
-        cmd,
-        cwd=os.getcwd(),
-        text=True
-    )
-    
-    logger.info(f"HSD started with PID {HSD_PROCESS.pid}")
+    try:
+        HSD_PROCESS = subprocess.Popen(
+            cmd,
+            cwd=os.getcwd(),
+            text=True
+        )
+        
+        logger.info(f"HSD started with PID {HSD_PROCESS.pid}")
+    except Exception as e:
+        logger.error(f"Failed to start HSD: {str(e)}", exc_info=True)
+        return
 
     atexit.register(hsdStop)
 
@@ -1959,6 +1965,19 @@ def hsdStart():
         logger.error(f"Failed to set signal handlers: {str(e)}", exc_info=True)
         pass
 
+def hsdRunning() -> bool:
+    global HSD_PROCESS
+    if not HSD_INTERNAL_NODE:
+        return False
+    if HSD_PROCESS is None:
+        return False
+    
+    # Check if process has terminated
+    poll_result = HSD_PROCESS.poll()
+    if poll_result is not None:
+        logger.error(f"HSD process has terminated with exit code: {poll_result}")
+        return False
+    return True
 
 def hsdStop():
     global HSD_PROCESS

example.env

@@ -4,4 +4,5 @@ THEME=black
 SHOW_EXPIRED=false
 EXPLORER_TX=https://shakeshift.com/transaction/
 DISABLE_WALLETDNS=false
-INTERNAL_HSD=false
+INTERNAL_HSD=false
+LOG_LEVEL=WARNING

main.py

@@ -7,13 +7,9 @@ from flask import Flask, make_response, redirect, request, jsonify, render_templ
 import os
 import dotenv
 import requests
-import account as account_module
-import render
 import re
 from flask_qrcode import QRcode
-import domainLookup
 import urllib.parse
-import plugin as plugins_module
 import gitinfo
 import datetime
 import time
@@ -22,18 +18,6 @@ from logging.handlers import RotatingFileHandler
 
 dotenv.load_dotenv()
 
-app = Flask(__name__)
-qrcode = QRcode(app)
-
-
-# Change this if network fees change
-fees = 0.02
-revokeCheck = random.randint(100000,999999)
-
-
-THEME = os.getenv("THEME", "black")
-
-
 # Setup logging
 if not os.path.exists('logs'):
     os.makedirs('logs')
@@ -42,15 +26,33 @@ handler = RotatingFileHandler(log_file, maxBytes=1024*1024, backupCount=3)
 formatter = logging.Formatter('[%(asctime)s] %(levelname)s in %(module)s: %(message)s')
 handler.setFormatter(formatter)
 logger = logging.getLogger()
-logger.setLevel(logging.WARNING)
+
+log_level = os.getenv("LOG_LEVEL", "WARNING").upper()
+if log_level in ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"]:
+    logger.setLevel(getattr(logging, log_level))
+else:
+    logger.setLevel(logging.WARNING)
 
 # Disable werkzeug logging
 logging.getLogger('werkzeug').setLevel(logging.INFO)
 logging.getLogger("urllib3").setLevel(logging.ERROR)
 logging.getLogger("requests").setLevel(logging.ERROR)
-
 logger.addHandler(handler)
 
+# Import after logger setup to capture logs from these modules
+import render  # noqa: E402
+import domainLookup  # noqa: E402
+import account as account_module  # noqa: E402
+import plugin as plugins_module  # noqa: E402
+
+app = Flask(__name__)
+qrcode = QRcode(app)
+
+# Change this if network fees change
+fees = 0.02
+revokeCheck = random.randint(100000,999999)
+THEME = os.getenv("THEME", "black")
+
 @app.route('/')
 def index():
     # Check if the user is logged in
@@ -315,11 +317,12 @@ def auctions():
         sort_time_next = reverseDirection(direction)
 
         # If older HSD version sort by domain height
-        if bids[0]['height'] == 0:
-            domains = sorted(domains, key=lambda k: k['height'],reverse=reverse)
-            sortbyDomain = True
-        else:
-            bids = sorted(bids, key=lambda k: k['height'],reverse=reverse)
+        if len(bids) > 0:
+            if bids[0]['height'] == 0:
+                domains = sorted(domains, key=lambda k: k['height'],reverse=reverse)
+                sortbyDomain = True
+            else:
+                bids = sorted(bids, key=lambda k: k['height'],reverse=reverse)
     else:
         # Sort by domain
         bids = sorted(bids, key=lambda k: k['name'],reverse=reverse)
@@ -1235,13 +1238,19 @@ def settings_action(action):
 
     if action == "logs":
         if not os.path.exists(log_file):
-            return jsonify({"error": "Log file not found"}), 404
+            return redirect("/settings?error=No log file found")
+        # Check if log file is empty
+        if os.path.getsize(log_file) == 0:
+            return redirect("/settings?error=Log file is empty")
+
         try:
             with open(log_file, 'rb') as f:
-                response = requests.put(f"https://upload.woodburn.au/{os.path.basename(log_file)}", data=f)
+                response = requests.put(f"https://upload.woodburn.au/{os.path.basename(log_file)}", data=f,
+                                        headers={"Max-Days": "5"})
             if response.status_code == 200 or response.status_code == 201:
-                url = response.text.strip().split('\n')[-1]
-                logger.info(f"Log upload successful: {url}")
+                token = response.text.strip().split('\n')[-1].split('/')[-2:]
+                url = f"https://upload.woodburn.au/inline/{token[0]}/{token[1]}"
+                logger.info(f"Log upload successful: {url}")                
                 return redirect(url)
             else:
                 logger.error(f"Failed to upload log: {response.status_code} {response.text}")
@@ -1859,9 +1868,43 @@ def api_icon(account):
 def api_status():
     # This doesn't require a login
     # Check if the node is connected
-    if not account_module.hsdConnected():
-        return jsonify({"status":503,"error": "Node not connected"}), 503
-    return jsonify({"status": 200,"result": "FireWallet is running"})
+    node_status = {
+        "connected": account_module.hsdConnected(),
+        "internal": account_module.HSD_INTERNAL_NODE,
+        "internal_running": False,
+        "version": "N/A"
+    }
+    status = 200
+    error = None
+    node_status['version'] = account_module.hsdVersion(False)
+    
+
+    if node_status['internal']:
+        node_status['internal_running'] = account_module.hsdRunning()
+    
+    # If the node is not connected, return an error
+    if not node_status['connected']:
+        error = "Node not connected"
+        status = 503
+    if node_status['version'] == -1:
+        error = "Error connecting to HSD"
+        status = 503
+    if node_status['internal'] and not node_status['internal_running']:
+        error = "Internal node not running"
+        status = 503
+    
+    commit = currentCurrentCommit()
+    return jsonify({
+        "node": node_status,
+        "version": {
+            "commit": commit,
+            "branch": currentCurrentBranch(),
+            "latest": runningLatestVersion(),
+            "url": f"https://git.woodburn.au/nathanwoodburn/firewalletbrowser/commit/{commit}" if commit != "Error" else None
+        },
+        "error": error,
+        "status": status
+    }), status
 
 
 #endregion
@@ -1906,6 +1949,45 @@ def renderDomain(name: str) -> str:
     except Exception:
         return f"{name}/"
 
+def currentCurrentCommit() -> str:
+    """
+    Get the current commit of the application.
+    """
+    if not os.path.exists(".git"):
+        return "Error"
+    info = gitinfo.get_git_info()
+    if info is None:
+        return "Error"
+    commit = info['commit']
+    return commit
+
+def currentCurrentBranch() -> str:
+    """
+    Get the current branch of the application.
+    """
+    if not os.path.exists(".git"):
+        return "Error"
+    info = gitinfo.get_git_info()
+    if info is None:
+        return "Error"
+    branch = info['refs']
+    return branch
+
+def runningLatestVersion() -> bool:
+    """
+    Check if the current version is the latest version.
+    """
+    if not os.path.exists(".git"):
+        return False
+    info = gitinfo.get_git_info()
+    if info is None:
+        return False
+    branch = info['refs']
+    commit = info['commit']
+    if commit != latestVersion(branch):
+        return False
+    return True
+
 def get_alerts(account:str) -> list:
     """
     Get alerts to show on the dashboard.
@@ -2090,8 +2172,7 @@ if __name__ == '__main__':
         logger.addHandler(console_handler)
         logger.setLevel(logging.DEBUG)
         app.run(debug=True, host=host, port=port)
-    else:
-        
+    else:        
         app.run(host=host, port=port)
 
 def tests():

templates/settings.html

@@ -133,7 +133,7 @@
                         <div class="card-body">
                             <h4 class="card-title">About</h4>
                             <h6 class="text-muted mb-2 card-subtitle">FireWallet is a UI to allow easy connection with HSD created by <a href="https://nathan.woodburn.au" target="_blank">Nathan.Woodburn/</a> and freely available. Please contact him <a href="https://l.woodburn.au/contact" target="_blank">here</a> if you would like to request any features or report any bugs.<br>FireWallet version: <code>{{version}}</code></h6>
-                            <div class="text-center"><a href="https://github.com/nathanwoodburn/firewalletbrowser" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration:none;" target="_blank"><i class="icon ion-social-github" style="color: var(--bs-emphasis-color);"></i>&nbsp;Github</a><a href="https://firewallet.au" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration:none;" target="_blank"><i class="icon ion-ios-information" style="color: var(--bs-emphasis-color);"></i>&nbsp;Website</a><a href="https://l.woodburn.au/donate" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration:none;" target="_blank"><i class="icon ion-social-usd" style="color: var(--bs-emphasis-color);"></i>&nbsp;Donate to support development</a><a href="/settings/logs" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration:none;" target="_blank"><i class="icon ion-help" style="color: var(--bs-emphasis-color);"></i>&nbsp;Upload logs for debugging</a></div>
+                            <div class="text-center"><a href="https://github.com/nathanwoodburn/firewalletbrowser" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration: none;display: inline-block;" target="_blank"><i class="icon ion-social-github" style="color: var(--bs-emphasis-color);"></i>&nbsp;Github</a><a href="https://firewallet.au" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration: none;display: inline-block;" target="_blank"><i class="icon ion-ios-information" style="color: var(--bs-emphasis-color);"></i>&nbsp;Website</a><a href="https://l.woodburn.au/donate" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration: none;display: inline-block;" target="_blank"><i class="icon ion-social-usd" style="color: var(--bs-emphasis-color);"></i>&nbsp;Donate to support development</a><a href="/settings/logs" style="margin: 15px;color: var(--bs-emphasis-color);text-decoration: none;display: inline-block;" target="_blank"><i class="icon ion-help" style="color: var(--bs-emphasis-color);"></i>&nbsp;Upload logs for debugging</a></div>
                         </div>
                     </div>
                 </div>