[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-server-cvs
Subject:    [Server-cvs] admin/web/build/proxy client.js, 1.2.14.1.8.1,
From:       svaidhya () helixcommunity ! org
Date:       2009-05-30 21:04:26
Message-ID: 20090530220902.58FA830ADAB () pukcab ! korelogic ! com
[Download RAW message or body]

Update of /cvsroot/server/admin/web/build/proxy
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv31785

Modified Files:
      Tag: SERVER_13_0
	client.js 
Log Message:

Synopsis
========
Admin system needs to universally accept (and add) brackets around IPv6 addresses


Branches:  SERVER_CURRENT, SERVER_13_0
Reviewer(s):  Chytanya


Description
===========

The server requires the IPv6 address to be enclosed within square brackets. But the \
admin system does not allow [] in any address.

There are 2 things to be taken care of:

1. If the IP Address is enclosed within [], the admin system should recognize it as a \
valid IP Address

   Made changes to a couple of places (common code used by all pages containing an IP \
Address input  field) to recognize [] as a valid IP v6 address.

2. If the an IPv6 address is not enclosed inside Sq bracket the admin system puts it \
within [] and  write to config file.

   Implementing this would take sometime. So for now if the user enters a valid IPv6 \
adress w/o  square bracktes, the admin sys would prompt the user to enclose IP \
Address within [].


Files Affected
==============
./server/admin/web/src/js/ip.js.wasm
./server/admin/web/src/js/client.js.wasm

HTML Files

./server/admin/web/build/(*)/ip.js
./server/admin/web/build/(*)/client.js


Testing Performed
=================
Unit Tests:
None.

Integration Tests:
- Verified the following

  - When an IPv6 Address WITH [] is entered, admin system accepts it and writes the \
address with []  in the config file.
  - When an IPv6 Address WITHOUT [] is entered, admin system  prompts the user to \
                enter it with []
  - When an IPv6 Address WITH MULTIPLE [] is entered, admin system complains about \
the IP being  invalid
  - When an IPv4 Address WITH [] is entered, admin system complains about the IP \
being  invalid
  - When an IPv4 Address WITHOUT [] is entered, admin system accepts it and writes \
the address  in the config file.

Leak Tests:
- None.

Performance Tests:
None.

Platforms Tested:  win32-i386-vc7
Build verified:    win32-i386-vc7 




Index: client.js
===================================================================
RCS file: /cvsroot/server/admin/web/build/proxy/client.js,v
retrieving revision 1.2.14.1.8.1
retrieving revision 1.2.14.1.8.2
diff -u -d -r1.2.14.1.8.1 -r1.2.14.1.8.2
--- client.js	11 May 2009 15:47:44 -0000	1.2.14.1.8.1
+++ client.js	30 May 2009 21:04:23 -0000	1.2.14.1.8.2
@@ -1078,7 +1078,8 @@
     if ( addr.bSearch( /\:/ ) )
 	{
 		// just hex digits, colons, dots and '/' ?
-		return addr.bSearch( /^ *[\dA-F\.\:\/]+ *$/i ); 
+        //Also allow v6 address enclosed winthin square brackets
+        return addr.bSearch( /^ *\[?[\dA-F\.\:\/]+\]? *$/i ); 
 	}
 	else
     {


_______________________________________________
Server-cvs mailing list
Server-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/server-cvs


[prev in list] [next in list] [prev in thread] [next in thread]