Merge pull request #1 from SuperDuperRob/master

Many tweaks and improvements :)

Author: sudofox

README.md

@@ -1,61 +1,47 @@
-# sudomemoDNS
-
-This server will allow you to connect to Sudomemo when
-your Internet Service Provider does not work with custom DNS.
+![sudomemoDNS Logo](/sudomemoDNS_banner.png)
+### This server will allow you to connect to Sudomemo when your Internet Service Provider does not work with custom DNS.
 
 ## Setup
 
-The setup process does not differ from what is shown at
-https://support.sudomemo.net/setup except for the values
-to enter in your custom DNS settings.
-
-First, make sure that your Nintendo DSi is connected to the
-same network as this computer.
+The setup process does not differ from what is shown at https://flipnot.es/setup except for the values to enter in your custom DNS settings.
+First, make sure that your console is connected to the same network as this computer.
+Please submit any issues to Sudomemo Support.
 
-This is the initial release of sudomemoDNS. Please submit any issues to Sudomemo Support.
-
-# Running on Windows:
+## Running on Windows:
 
 Run the .exe provided in this release. You may have to click past a warning from Windows SmartScreen (as it sometimes turns its nose up at applications bundled by PyInstaller) or your firewall (as it needs to listen for DNS requests).
 
-# Running on anything else:
+## Running on anything else:
 
 Required packages (installable with pip): requests, dnslib
 Required Python version: 3+, tested with 3.6
 
     sudo <name of your python binary> sudomemoDNS.py
 
-# What you'll see when it starts up
+## What you'll see when it starts up
 
 The following will display when you start it up:
 
     +===============================+
     |      Sudomemo DNS Server      |
-    |          Version 1.0          |
+    |          Version 1.1          |
     +===============================+
-    
-    Hello! This server will allow you to connect to Sudomemo when
-    your Internet Service Provider does not work with custom DNS.
-
-    #### How To Use ####
-    
-    The setup process does not differ from what is shown at
-    https://support.sudomemo.net/setup except for the values
-    to enter in your custom DNS settings.
-    
-    First, make sure that your Nintendo DSi is connected to the
-    same network as this computer.
-    
-    Here are the settings you will put in for DNS on your Nintendo DSi:
-    
-    Primary DNS: XX.XX.XX.XX (NOTE: this value will be unique when you run the program)
+
+    == Welcome to sudomemoDNS! ==
+    This server will allow you to connect to Sudomemo when your Internet Service Provider does
+    not work with custom DNS.
+
+    == How To Use ==
+    First, make sure that your console is connected to the same network as this computer.
+
+    Then, put these settings in for DNS on your console:
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+    Primary DNS:   XXX.XXX.XXX.XXX (NOTE: This value will be unique when you run the program)
     Secondary DNS: 8.8.8.8
-    All other settings should match what is shown at the above URL.
-    
-    #### Getting Help ####
-    
-    Need help? Visit our Discord server or check out https://support.sudomemo.net.
+    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
+    == Getting Help ==
+    Need help? Visit our Discord server or check out https://support.sudomemo.net
 
 When entering your DNS settings, if the Primary DNS is displayed like this (this is an example):
 
@@ -65,10 +51,9 @@ then you can enter it as follows:
 
 192.168.001.007
 
+## Building on Windows
 
-# Building on Windows
-
-Install dnslib and requests via pip (as well as pyinstaller) and then build with pyinstaller:
+Install dnslib, requests, and pyinstaller via pip, then modify the paths in the spec file to where sudomemoDNS and the sudomemoDNS icon is stored.
+Then, run the following as an administrator:
 
-C:\Users\Administrator\sudomemo-dns>C:\Users\Administrator\AppData\Local\Programs\Python\Python36\Scripts\pyinstaller.exe sud
-omemoDNS_v1.0.spec
+    pyinstaller.exe sudomemoDNS_v1.1.spec

sudomemo.ico

@@ -1,5 +1,5 @@
-# sudomemoDNS v1.0
-# (c) 2019 Austin Burk/Sudomemo
+# sudomemoDNS v1.1
+# (c) 2019-2020 Austin Burk / Team Sudomemo
 # All rights reserved
 
 from datetime import datetime
@@ -18,15 +18,15 @@ def get_platform():
     platforms = {
         'linux1' : 'Linux',
         'linux2' : 'Linux',
-        'darwin' : 'OS X',
+        'darwin' : 'macOS',
         'win32' : 'Windows'
     }
     if sys.platform not in platforms:
         return sys.platform
 
     return platforms[sys.platform]
 
-SUDOMEMODNS_VERSION = "1.0"
+SUDOMEMODNS_VERSION = "1.1"
 
 def get_ip():
     s = socket.socket(socket.AF_INET, socket.SOCK_DGRAM)
@@ -49,27 +49,22 @@ def get_ip():
 print("|          Version " + SUDOMEMODNS_VERSION + "          |")
 print("+===============================+\n")
 
-print("Hello! This server will allow you to connect to Sudomemo when")
-print("your Internet Service Provider does not work with custom DNS.\n")
+print("== Welcome to sudomemoDNS! ==") 
+print("This server will allow you to connect to Sudomemo when your Internet Service Provider does not work with custom DNS.\n")
 
-print("#### How To Use ####\n")
-print("The setup process does not differ from what is shown at")
-print("https://support.sudomemo.net/setup except for the values")
-print("to enter in your custom DNS settings.\n")
+print("== How To Use ==")
+print("First, make sure that your console is connected to the same network as this computer.\n")
 
-print("First, make sure that your Nintendo DSi is connected to the")
-print("same network as this computer.")
-
-print("\nHere are the settings you will put in for DNS on your Nintendo DSi:\n")
+print("Then, put these settings in for DNS on your console:")
+print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
 print("Primary DNS:  ",MY_IP)
 print("Secondary DNS: 8.8.8.8")
+print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n")
 
-print("All other settings should match what is shown at the above URL.\n")
-
-print("#### Getting Help ####\n")
-print("Need help? Visit our Discord server or check out https://support.sudomemo.net.\n")
+print("== Getting Help ==")
+print("Need help? Visit our Discord server or check out https://support.sudomemo.net\n")
 
-print("[INFO] Starting up")
+print("[INFO] Starting sudomemoDNS...")
 
 TYPE_LOOKUP = {
     A: QTYPE.A,
@@ -89,17 +84,16 @@ def log_recv(self, handler, data):
     def log_send(self, handler, data):
         pass
     def log_request(self, handler, request):
-        print("[INFO] Received DNS request from DSi at " + handler.client_address[0])
+        print("[INFO] Received DNS request from console at " + handler.client_address[0])
     def log_reply(self, handler, reply):
-        print("[INFO] Sent response to DSi at " + handler.client_address[0])
+        print("[INFO] Sent response to console at " + handler.client_address[0])
     def log_error(self, handler, e):
-        logger.error("[INFO] Invalid DNS request from " + handler.client_address[0])
+        print("[ERROR] Invalid DNS request from " + handler.client_address[0])
     def log_truncated(self, handler, reply):
         pass
     def log_data(self, dnsobj):
         pass
 
-
 class Record:
     def __init__(self, rdata_type, *args, rtype=None, rname=None, ttl=None, **kwargs):
         if isinstance(rdata_type, RD):
@@ -154,17 +148,15 @@ def __str__(self):
 try:
   get_zones = requests.get("https://www.sudomemo.net/api/dns_zones.json", headers={'User-Agent': 'SudomemoDNS/' + SUDOMEMODNS_VERSION + ' (' + get_platform() + ')'})
 except requests.exceptions.Timeout:
-  print("[ERROR] Couldn't load DNS data: connection to Sudomemo timed out.")
-  print("[ERROR] Are you connected to the Internet?")
+  print("[ERROR] Unable to load DNS data: Connection to Sudomemo timed out. Are you connected to the Internet?")
 except requests.exceptions.RequestException as e:
-  print("[ERROR] Couldn't load DNS data.")
+  print("[ERROR] Unable load DNS data.")
   print("[ERROR] Exception: ",e)
   sys.exit(1)
 try:
   zones = json.loads(get_zones.text)
 except ValueError as e:
-  print("[ERROR] Couldn't load DNS data: invalid response from server")
-  print("[ERROR] Check that you can visit sudomemo.net")
+  print("[ERROR] Unable load DNS data: Invalid response from server. Check that you can visit sudomemo.net")
 
 for zone in zones:
   if zone["type"] == "a":
@@ -203,35 +195,31 @@ def resolve(self, request, handler):
 resolver = Resolver()
 dnsLogger = SudomemoDNSLogger()
 
-print("[INFO] Detected operating system:", get_platform());
+print("[INFO] Detected operating system:", get_platform())
 
 if get_platform() == 'linux':
   print("[INFO] Please note that you will have to run this as root or with permissions to bind to UDP port 53.")
   print("[INFO] If you aren't seeing any requests, check that this is the case first with lsof -i:53 (requires lsof)")
   print("[INFO] To run as root, prefix the command with 'sudo'")
-elif get_platform() == 'OS X':
+elif get_platform() == 'macOS':
   print("[INFO] Please note that you will have to run this as root or with permissions to bind to UDP port 53.")
   print("[INFO] If you aren't seeing any requests, check that this is the case first with lsof -i:53 (requires lsof)")
   print("[INFO] To run as root, prefix the command with 'sudo'")
 elif get_platform() == 'Windows':
-  print("[INFO] Please note: On Windows, you may have to allow this application through")
-  print("[INFO] the firewall. If so, a popup will appear in a moment.")
-  print("[INFO] If you aren't seeing any requests, make sure you have done so first.")
-  print("[INFO] Disregard this message if you have already done so.")
+  print("[INFO] Please note that you may have to allow this application through the firewall. If so, a popup will appear in a moment.")
+  print("[INFO] If you are not seeing any requests, make sure you have allowed this application through the firewall. If you have already done so, disregard this message.")
 
 try:
   servers = [
     DNSServer(resolver=resolver, port=53, address=MY_IP, tcp=True, logger=dnsLogger),
     DNSServer(resolver=resolver, port=53, address=MY_IP, tcp=False, logger=dnsLogger),
   ]
 except PermissionError:
-  print("[ERROR] Permission error: check that you are running this as Administrator or root")
+  print("[ERROR] Permission error: Check that you are running this as an administrator or root")
   sys.exit(1)
 
 
-print("[INFO] Starting Sudomemo DNS server.")
-
-print("[INFO] Now waiting for DNS requests from your Nintendo DSi System.")
+print("[INFO] sudomemoDNS is ready. Now waiting for DNS requests from your console...")
 
 if __name__ == '__main__':
     for s in servers:
@@ -244,4 +232,4 @@ def resolve(self, request, handler):
         pass
     finally:
         for s in servers:
-            s.stop()
+            s.stop()

sudomemoDNS.py

@@ -4,7 +4,7 @@ block_cipher = None
 
 
 a = Analysis(['sudomemoDNS.py'],
-             pathex=['C:\\Users\\Administrator\\sudomemo-dns'],
+             pathex=['C:\\PATH\\TO\\SUDOMEMODNS'],
              binaries=[],
              datas=[],
              hiddenimports=[],
@@ -23,10 +23,10 @@ exe = EXE(pyz,
           a.zipfiles,
           a.datas,
           [],
-          name='sudomemoDNS_v1.0',
+          name='sudomemoDNS_v1.1',
           debug=False,
           bootloader_ignore_signals=False,
           strip=False,
           upx=True,
           runtime_tmpdir=None,
-          console=True , icon='C:\\Users\\Administrator\\sudomemo-dns\\sudomemo.ico')
+          console=True , icon='C:\\PATH\\TO\\SUDOMEMODNS\\ICON.ico')