separate network status and errors

daemon/HTTPServer.cpp

@@ -232,25 +232,24 @@ namespace http {
 			case eRouterStatusUnknown: s << tr("Unknown"); break;
 			case eRouterStatusProxy: s << tr("Proxy"); break;
 			case eRouterStatusMesh: s << tr("Mesh"); break;
-			case eRouterStatusError:
+			default: s << tr("Unknown");
+		}
+		if (error != eRouterErrorNone)
+		{
+			s << "<br>";
+			switch (error)
 			{
-				s << tr("Error");
+				case eRouterErrorClockSkew:
-				switch (error)
+					s << " - " << tr("Clock skew");
-				{
-					case eRouterErrorClockSkew:
-						s << " - " << tr("Clock skew");
-					break;
-					case eRouterErrorOffline:
-						s << " - " << tr("Offline");
-					break;
-					case eRouterErrorSymmetricNAT:
-						s << " - " << tr("Symmetric NAT");
-					break;
-					default: ;
-				}
 				break;
+				case eRouterErrorOffline:
+					s << " - " << tr("Offline");
+				break;
+				case eRouterErrorSymmetricNAT:
+					s << " - " << tr("Symmetric NAT");
+				break;
+				default: ;
 			}
-			default: s << tr("Unknown");
 		}	
 	}
 

libi2pd/RouterContext.h

@@ -37,10 +37,9 @@ namespace garlic
 		eRouterStatusOK = 0,
 		eRouterStatusTesting = 1,
 		eRouterStatusFirewalled = 2,
-		eRouterStatusError = 3,
+		eRouterStatusUnknown = 3,
-		eRouterStatusUnknown = 4,
+		eRouterStatusProxy = 4,
-		eRouterStatusProxy = 5,
+		eRouterStatusMesh = 5
-		eRouterStatusMesh = 6
 	};
 
 	enum RouterError
@@ -105,12 +104,12 @@ namespace garlic
 			void SetStatus (RouterStatus status);
 			void SetStatusSSU2 (RouterStatus status);
 			RouterError GetError () const { return m_Error; };
-			void SetError (RouterError error) { m_Status = eRouterStatusError; m_Error = error; };
+			void SetError (RouterError error) { m_Error = error; };
 			RouterStatus GetStatusV6 () const { return m_StatusV6; };
 			void SetStatusV6 (RouterStatus status);
 			void SetStatusV6SSU2 (RouterStatus status);
 			RouterError GetErrorV6 () const { return m_ErrorV6; };
-			void SetErrorV6 (RouterError error) { m_StatusV6 = eRouterStatusError; m_ErrorV6 = error; };
+			void SetErrorV6 (RouterError error) { m_ErrorV6 = error; };
 			int GetNetID () const { return m_NetID; };
 			void SetNetID (int netID) { m_NetID = netID; };
 			bool DecryptTunnelBuildRecord (const uint8_t * encrypted, uint8_t * data);

libi2pd/SSU2Session.cpp

@@ -1571,13 +1571,13 @@ namespace transport
 				{
 					if (isV4)
 					{
-						if (i2p::context.GetStatus () == eRouterStatusError && i2p::context.GetError () == eRouterErrorSymmetricNAT)
+						if (i2p::context.GetError () == eRouterErrorSymmetricNAT)
-							i2p::context.SetStatus (eRouterStatusTesting);
+							i2p::context.SetError (eRouterErrorNone);
 					}	
 					else
 					{
-						if (i2p::context.GetStatusV6 () == eRouterStatusError && i2p::context.GetErrorV6 () == eRouterErrorSymmetricNAT)
+						if (i2p::context.GetErrorV6 () == eRouterErrorSymmetricNAT)
-							i2p::context.SetStatusV6 (eRouterStatusTesting);
+							i2p::context.SetErrorV6 (eRouterErrorNone);
 					}		
 				}	
 			}	

libi2pd/SSUSession.cpp

@@ -719,8 +719,8 @@ namespace transport
 				if (i2p::context.GetStatus () == eRouterStatusTesting)
 					i2p::context.SetError (eRouterErrorSymmetricNAT);
 			}
-			else if (i2p::context.GetStatus () == eRouterStatusError && i2p::context.GetError () == eRouterErrorSymmetricNAT)
+			else if (i2p::context.GetError () == eRouterErrorSymmetricNAT)
-				i2p::context.SetStatus (eRouterStatusTesting);
+				i2p::context.SetError (eRouterErrorNone);
 		}
 		uint32_t nonce = bufbe32toh (buf);
 		buf += 4; // nonce