fix issue8: set NmapProcess.summary as deprecated (see changelog)

savon-noir · savon-noir · commit 4a2f9cd3e1e0 · 2013-12-26T19:18:35.000+01:00
diff --git a/CHANGES.txt b/CHANGES.txt
@@ -1,3 +1,10 @@
+v0.4.2, 26/10/2013 -- Bug corrected: issue 8
+                      There is no guarantee that "finished" or "runstats" will be
+                      received by event parser of process.py.
+                      Summary functions are now flagged as deprecated. To use data
+                      from scan summary of numbers of hosts up, the user of the
+                      lib will have to use NmapParser.parse() and the appropriate
+                      accessors.
 v0.4.1, 26/10/2013 -- Bug corrected: issue6
                       Infinite loop while launching several nmap scans in background
 v0.4.0, 28/10/2013 -- Bug corrected in missing data from nmap scan output
diff --git a/libnmap/process.py b/libnmap/process.py
@@ -6,6 +6,7 @@
 import threading
 from threading import Thread
 from xml.dom import pulldom
+import warnings
 try:
     from Queue import Queue, Empty, Full
 except ImportError:
@@ -423,6 +424,7 @@ def endtime(self):
 
         :return: string. Unix timestamp
         """
+        warnings.warn("data collected from finished events are deprecated. Use NmapParser.parse()", DeprecationWarning)
         return self.__endtime
 
     @property
@@ -432,6 +434,7 @@ def elapsed(self):
 
         :return: string
         """
+        warnings.warn("data collected from finished events are deprecated. Use NmapParser.parse()", DeprecationWarning)
         return self.__elapsed
 
     @property
@@ -441,6 +444,7 @@ def summary(self):
 
         :return: string
         """
+        warnings.warn("data collected from finished events are deprecated. Use NmapParser.parse()", DeprecationWarning)
         return self.__summary
 
     @property
