v1.0.1 - Slight cleanup and documentation- AI assisted.

main.py

@@ -2,40 +2,40 @@ from enum import Enum
 from dataclasses import dataclass, field
 import time
 import ipaddress
-import dns
 import fqdn
 
-class InvaliadDataException(Exception):
+class InvalidDataException(Exception):
-	"""Exception raied when invaliad data is passed to a record"""
+	"""Exception raised when invalid data is passed to a record."""
 
 	def __init__(self, message):
-		self.message	= message
+		self.message = message
 		super().__init__(self, message)
 
 @dataclass
 class Record:
-	def __int__(self, name: str = '@', ttl: str = 3600, rtype: str = 'A', data: str = '0.0.0.0'):
+	"""Base class for DNS records."""
-		self.rtype: str	= rtype
+
-		self.name	= name
+	rclass: str	= 'IN'    # DNS class, usually 'IN' for internet
-		self.data	= data
+	rtype: str	= 'A'      # Record type (A, AAAA, MX, etc.)
-		self.ttl	= ttl
+	name: str	= '@'       # Name of the record (e.g., domain name)
+	data: str	= '0.0.0.0' # Data associated with the record (e.g., IP address or hostname)
+	ttl: int	= 3600       # Time to live (TTL) for the record in seconds
 
 	def __str__(self):
+		"""Returns a string representation of the record."""
 		return f"{self.name} {self.ttl} {self.rclass} {self.rtype} {self.data}"
 
-	rclass: str = 'IN'
-	rtype: str	= 'A'
-	name: str	= '@'
-	data: str	= '0.0.0.0'
-	ttl: int	= 6400
-
 @dataclass
 class A(Record):
-	def __init__(self, name: str = '@', ttl: str = 3600, target: str = '0.0.0.0'):
+	"""Represents an 'A' (IPv4 address) record."""
-		if isinstance(ipaddress.ip_address(data), ipaddress.IPv4Address):
+
-			self.data = data
+	#host: str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, host: str = '0.0.0.0'):
+		if isinstance(ipaddress.ip_address(host), ipaddress.IPv4Address):
+			self.data = host
 		else:
-			raise InvaliadDataException(message=f'{data} is not a valid IPv4 Address.')
+			raise InvalidDataException(message=f'{host} is not a valid IPv4 address.')
 
 		self.rtype	= 'A'
 		self.name	= name
@@ -43,11 +43,15 @@ class A(Record):
 
 @dataclass
 class AAAA(Record):
-	def __init__(self, name: str = '@', ttl: str = 3600, target: str = '0.0.0.0'):
+	"""Represents an 'AAAA' (IPv6 address) record."""
-		if isinstance(ipaddress.ip_address(data), ipaddress.IPv4Address):
+
-			self.data = data
+	#host:str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, host: str = '0.0.0.0'):
+		if isinstance(ipaddress.ip_address(host), ipaddress.IPv6Address):
+			self.data = host
 		else:
-			raise InvaliadDataException(message=f'{data} is not a valiad IPv6 Address.')
+			raise InvalidDataException(message=f'{host} is not a valid IPv6 address.')
 
 		self.rtype	= 'AAAA'
 		self.name	= name
@@ -55,155 +59,210 @@ class AAAA(Record):
 
 @dataclass
 class CNAME(Record):
-	def __init__(self, name: str = '@', ttl: str = 3600, target: str = '0.0.0.0'):
+	"""Represents a 'CNAME' (Canonical Name) record."""
+
+	#target: str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, target: str = 'example.com'):
 		self.rtype	= 'CNAME'
 		self.name	= name
 		self.ttl	= ttl
 
-		if(fqdn.FQDN(target).is_valid):
+		if fqdn.FQDN(target).is_valid:
 			self.data = target
 		else:
-			raise InvaliadDataException
+			raise InvalidDataException(message=f'{target} is not a valid FQDN')
 
 @dataclass
 class MX(Record):
-	def __init__(self, name: str = '@', ttl: str = 3600, priority: int = '10', host: str = 'example.com'):
+	"""Represents an 'MX' (Mail Exchange) record."""
+
+	#host: str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, priority: int = 10, host: str = 'example.com'):
 		self.rtype	= 'MX'
 		self.name	= name
 		self.ttl	= ttl
 		self.priority	= priority
-		self.data	= "{ttl} {priority}"
+		self.host = host
+		self.data = f"{self.priority} {self.host}"
 
-		if(fqdn.FQDN(host).is_valid):
+		if fqdn.FQDN(host).is_valid:
-			self.host = host
+			self.data = f"{self.priority} {self.host}"
 		else:
-			raise InvaliadDataException(message='{host} is not a valid FQDN')
+			raise InvalidDataException(message=f'{host} is not a valid FQDN')
 
 @dataclass
 class NS(Record):
-	def __init__(self, name: str = '@', ttl: str = 3600, target: str = 'example.com'):
+	"""Represents an 'NS' (Name Server) record."""
-		self.rtype	= 'MX'
+
+	#target: str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, target: str = 'example.com'):
+		self.rtype	= 'NS'
 		self.name	= name
 		self.ttl	= ttl
+		self.target	= target
 
-		if (fqdn.FQDN(host).is_valid):
+		if fqdn.FQDN(target).is_valid:
-			self.host = host
+			self.data = target
 		else:
-			raise InvaliadDataException(message='{target} is nod a valid FQDN')
+			raise InvalidDataException(message=f'{target} is not a valid FQDN')
-		self.data	= target
 
 @dataclass
 class PTR(Record):
-	def __init__(self, name: str = '@', ttl: str = 3600, host: str = '0.0.0.0'):
+	"""Represents a 'PTR' (Pointer) record."""
+
+	#host: str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, host: str = 'example.com'):
 		self.rtype	= 'PTR'
 		self.name	= name
 		self.ttl	= ttl
 
-		if(fqdn.FQDN(host).is_valid):
+		if fqdn.FQDN(host).is_valid:
 			self.data = host
 		else:
-			raise InvaliadDataException(message='{host} is not a valid FQDN')
+			raise InvalidDataException(message=f'{host} is not a valid FQDN')
 
-# TODO: Cleanup. I have no idea why I have _str_() defined.
 @dataclass
 class SOA(Record):
+	"""Represents an 'SOA' (Start of Authority) record."""
+
+	#mname: str
+	#rname: str
+	#serial: int
+	#refresh: int
+	#retry: int
+	#expire: int
+
 	def __init__(self, name: str = '@', mname: str = 'ns1.example.com', rname: str = 'admin.example.com',
 				serial: int = int(time.time()), refresh: int = 86400, retry: int = 7200,
-				expire: int = 15552000, ttl: int = 21700
+				expire: int = 15552000, ttl: int = 21700):
-		):
+		self.rtype	= 'SOA'
-		self.rtype		= "SOA"
+		self.name	= name
-		self.name		= name
+		self.mname	= mname
-		self.mname		= mname
+		self.rname	= rname
-		self.rname		= rname
+		self.serial	= serial
-		self.serial		= serial
 		self.refresh	= refresh
-		self.retry		= retry
+		self.retry	= retry
-		self.expire		= expire
+		self.expire	= expire
-		self.ttl		= ttl
+		self.ttl	= ttl
-		self.data		= "{name} {ttl} IN SOA {mname} {rname} {serial} {refresh} {retry} {expire} {ttl}"
+		self.data	= f"{self.name} {self.ttl} IN SOA {self.mname} {self.rname} {self.serial} {self.refresh} {self.retry} {self.expire}"
 
 @dataclass
 class SRV(Record):
-	def __int__(self, name: str = '@', ttl: str = 3600, service: str = "service", protocol: str = 'proto',
+	"""Represents an 'SRV' (Service) record."""
-				priority: int = 10, weight: int = 10, port: int = 0, target: str = 'example.com'
+
-				):
+	#service: str
-		self.rtype	= 'PTR'
+	#protocol: str
-		self.name	= '_{service}._{protocol}.name'
+	#priority: int
-		self.ttl 	= ttl
+	#weight: int
+	#port: int
+	#target: str
+
+	def __init__(self, name: str = '@', ttl: int = 3600, service: str = "service", protocol: str = 'proto',
+				priority: int = 10, weight: int = 10, port: int = 0, target: str = 'example.com'):
+		self.rtype	= 'SRV'
+		self.name	= f"_{service}._{protocol}.{name}"
+		self.ttl	= ttl
 		self.service	= service
 		self.protocol	= protocol
 		self.priority	= priority
-		self.weight		= weight
+		self.weight	= weight
 		self.port	= port
-		if (fqdn.FQDN(host).is_valid):
+
-			self.target	= target
+		if fqdn.FQDN(target).is_valid:
+			self.target = target
 		else:
-			raise InvaliadDataException(message='{target} is not a valiad FQDN')
+			raise InvalidDataException(message=f'{target} is not a valid FQDN')
-		self.data	= "{priority} {weight} {port} {target}"
+
+		self.data = f"{self.priority} {self.weight} {self.port} {self.target}"
 
 @dataclass
 class Zone:
+	"""Represents a DNS zone containing multiple records."""
+
 	origin: str
 	records: list = field(default_factory=list)
 
+	def __init__(self, origin: str):
+		"""Initializes a zone with the given origin and ensures it ends with a dot."""
+		if origin[-1] != '.':
+			self.origin = origin + '.'
+		else:
+			self.origin = origin
+
 	def __str__(self):
-		zone: str = ''
+		"""Returns a string representation of the zone."""
-
+		zone = ''
 		for record in self.records:
-			zone.join(record)
+			zone += str(record) + '\n'
-
 		return zone
 
-	def __mkfqdn(self, name):
+	def __mkfqdn(self, name: str) -> str:
+		"""Converts a name to a fully qualified domain name (FQDN)."""
 		if name[-1] != '.':
 			return name + '.' + self.origin
 		else:
 			return name
 
 	def new_A(self, name: str = '@', ttl: int = 3600, data: str = '0.0.0.0'):
+		"""Creates and adds a new A record to the zone."""
 		name = self.__mkfqdn(name)
 		self.add(A(name=name, ttl=ttl, data=data))
 
 	def new_AAAA(self, name: str = '@', ttl: int = 3600, data: str = '0.0.0.0'):
+		"""Creates and adds a new AAAA record to the zone."""
 		name = self.__mkfqdn(name)
 		self.add(AAAA(name=name, ttl=ttl, data=data))
 
-	def new_CNAME(self, name: str = '@', ttl: int = 3600, data: str = '0.0.0.0'):
+	def new_CNAME(self, name: str = '@', ttl: int = 3600, data: str = 'example.com'):
+		"""Creates and adds a new CNAME record to the zone."""
 		name = self.__mkfqdn(name)
-		self.add(CNAME(name=name, ttl=ttl, data=data))
+		self.add(CNAME(name=name, ttl=ttl, target=data))
 
 	def new_MX(self, name: str = '@', ttl: int = 3600, priority: int = 10, host: str = 'example.com'):
+		"""Creates and adds a new MX record to the zone."""
 		name = self.__mkfqdn(name)
 		self.add(MX(name=name, ttl=ttl, priority=priority, host=host))
 
 	def new_NS(self, name: str = '@', ttl: int = 3600, target: str = 'example.com'):
+		"""Creates and adds a new NS record to the zone."""
 		name = self.__mkfqdn(name)
 		self.add(NS(name=name, ttl=ttl, target=target))
 
 	def new_PTR(self, name: str = '@', ttl: int = 3600, host: str = 'example.com'):
+		"""Creates and adds a new PTR record to the zone."""
 		name = self.__mkfqdn(name)
 		self.add(PTR(name=name, ttl=ttl, host=host))
 
-	def new_soa(self, mname: str = 'ns1.example.com', rname: str = 'admin.example.com', serial: int = int(time.time()), refresh: int = 86400, retry: int = 7200, expire: int = 15552000, ttl: int = 21700):
+	def new_soa(self, mname: str = 'ns1.example.com', rname: str = 'admin.example.com',
-		mname = self.__mkfqdn(name)
+				serial: int = int(time.time()), refresh: int = 86400, retry: int = 7200,
+				expire: int = 15552000, ttl: int = 21700):
+		"""Creates and adds a new SOA record to the zone."""
+		mname = self.__mkfqdn(mname)
 		self.add(SOA(mname=mname, rname=rname, serial=serial, refresh=refresh, retry=retry, expire=expire, ttl=ttl))
 
-	def new_SRV(self, name: str = '@', ttl: int = 3600, service: str = 'service', protocol: str = 'proto', priority: int = 10, weight: int = 10, target: str = 'example.com'):
+	def new_SRV(self, name: str = '@', ttl: int = 3600, service: str = 'service', protocol: str = 'proto',
+				priority: int = 10, weight: int = 10, target: str = 'example.com'):
+		"""Creates and adds a new SRV record to the zone."""
 		name = self.__mkfqdn(name)
-		self.add(SRV(name=name, ttl=ttl, target=target))
+		self.add(SRV(name=name, ttl=ttl, service=service, protocol=protocol,
-
+				priority=priority, weight=weight, target=target))
 
 	def new_record(self, name: str = '@', ttl: int = 3600, rtype: str = 'A', data: str = '0.0.0.0'):
+		"""Creates and adds a generic DNS record to the zone."""
 		name = self.__mkfqdn(name)
-		self.add(name=name, ttl=ttl, rtype=rtype, data=data)
+		self.add(Record(name=name, ttl=ttl, rtype=rtype, data=data))
 
 	def add(self, record: Record):
+		"""Adds a record to the zone."""
 		self.records.append(record)
 
 	def save_file(self, filepath: str):
+		"""Saves the zone records to a file."""
 		with open(filepath, 'w') as file:
 			for record in self.records:
 				file.write(str(record) + '\n')
 				print(str(record))
-		file.close()
-