Changes between Version 5 and Version 6 of MapGuideRfc118

Timestamp:

07/04/11 02:28:12 (13 years ago)

Author:

wuma

Comment:

--

MapGuideRfc118

@@ -111 +111 @@
 }}}
 
+ 5. '''Change the way encoding Site IpAddress into SessionId'''
+MapGuide Server encodes the IP address of site server into the Session Id so that it could pick the right site server from the list by comparing the IP address.
+The encoding logic currently is: '''{{{(INTERNAL_API)}}}'''
+{{{
+STRING MgSiteInfo::ToHexString()
+{
+    STRING hexString;
+    UINT32 n1, n2, n3, n4;
+    wchar_t buffer[30];
+    if (4 == ::swscanf(m_target.c_str(), L"%u.%u.%u.%u", &n1, &n2, &n3, &n4))
+    {
+        swprintf(buffer, 30, L"%.2X%.2X%.2X%.2X%.4X%.4X%.4X", n1, n2, n3, n4, 
+                 m_sitePort, m_clientPort, m_adminPort);
+        hexString = buffer;
+    }
+
+    return hexString;
+} 
+}}}
+Then it works for only IPv4. And for IPv6, because the valid format is really complicated, then convert it to a hex string is difficult.
+The solution is: encode the IP as Base64 string. The port numbers will still be converted to hex string so that they could be easily decoded.
+Then the new method will be like: 
+{{{
+STRING MgSiteInfo::ToHexString()
+{
+    STRING hexString;
+    
+    char buf[100] = {0};
+    char* target = ACE_Wide_To_Ascii(m_target.c_str()).char_rep();
+    Base64::Encode(buf, (unsigned char*)target, (unsigned long)strlen(target));
+    
+    wchar_t buffer[100] = {0};  
+    swprintf(buffer, L"%s%.4X%.4X%.4X", ACE_Ascii_To_Wide(buf).wchar_rep(),     
+             m_sitePort, m_clientPort, m_adminPort);
+    hexString = buffer;
+
+    return hexString;
+}
+}}}
+One thing that needs attention is: A Base64 string might have “=” appended at the end for alignment. But “=” is treated as a reserved character. The solutions is: we remove all appending “=” when doing encoding, and then append them back when doing decoding
+
 == Implications ==
-
+Since it just makes MGOS work with IPv6, it doesn't change any existing functions. 
 
 
 == Test Plan ==
-
+Since it doesn't change any existing functions, so no new test cases need be added.