Enable parsing of IPv6 form addresse (see rfc2373 and rfc2732).

sdk/lib/uri/uri.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library dart.uri;
 
 import 'dart:math';
 import 'dart:utf';
 
 part 'encode_decode.dart';
 part 'helpers.dart';
 
 /**
  * A parsed URI, inspired by Closure's [URI][] class. Implements [RFC-3986][].
+ * The domain component can either be a hostname, a IPv4 address or an IPv6
+ * address, contained in '[' and ']', following [RFC-2732][]. If the domain
+ * component contains a ':', the String returned from [toString] will have
+ * '[' and ']' around the part.

[floitsch, 2013/05/01 08:53:30]

which part?

[Anders Johnsen, 2013/05/01 09:26:14]

Done.

  * [uri]: http://closure-library.googlecode.com/svn/docs/class_goog_Uri.html

[floitsch, 2013/05/01 08:53:30]

URI ?

[Anders Johnsen, 2013/05/01 09:26:14]

Done.

  * [RFC-3986]: http://tools.ietf.org/html/rfc3986#section-4.3)
+ * [RFC-2732]: http://www.ietf.org/rfc/rfc2732.txt
  */
 class Uri {
   final String scheme;
   final String userInfo;
   final String domain;
   final int port;
   final String path;
   final String query;
   final String fragment;
 
   /**
    * Deprecated. Please use [parse] instead.

[floitsch, 2013/05/01 08:53:30]

Now that I see it: yes. Please get rid of it.

[Anders Johnsen, 2013/05/01 09:26:14]

Done.

    */
   Uri.fromString(String uri) : this._fromMatch(_splitRe.firstMatch(uri));
 
   static Uri parse(String uri) => new Uri._fromMatch(_splitRe.firstMatch(uri));
 
   Uri._fromMatch(Match m) :
     this.fromComponents(scheme: _emptyIfNull(m[_COMPONENT_SCHEME]),
                         userInfo: _emptyIfNull(m[_COMPONENT_USER_INFO]),
-                        domain: _emptyIfNull(m[_COMPONENT_DOMAIN]),
+                        domain: _eitherOf(
+                            m[_COMPONENT_DOMAIN], m[_COMPONENT_DOMAIN_IPV6]),
                         port: _parseIntOrZero(m[_COMPONENT_PORT]),
                         path: _emptyIfNull(m[_COMPONENT_PATH]),
                         query: _emptyIfNull(m[_COMPONENT_QUERY_DATA]),
                         fragment: _emptyIfNull(m[_COMPONENT_FRAGMENT]));
 
   const Uri.fromComponents({this.scheme: "",
                             this.userInfo: "",
                             this.domain: "",
                             this.port: 0,
                             this.path: "",
                             this.query: "",
                             this.fragment: ""});
 
   Uri(String uri) : this.fromString(uri);
 
   static String _emptyIfNull(String val) => val != null ? val : '';
 
   static int _parseIntOrZero(String val) {
     if (val != null && val != '') {
       return int.parse(val);
     } else {
       return 0;
     }
   }
 
+  static String _eitherOf(String val1, String val2) {
+    if (val1 != null) return val1;
+    if (val2 != null) return val2;
+    return '';
+  }
+
   // NOTE: This code was ported from: closure-library/closure/goog/uri/utils.js
   static final RegExp _splitRe = new RegExp(
       '^'
       '(?:'
         '([^:/?#.]+)'                   // scheme - ignore special characters
                                         // used by other URL parts such as :,
                                         // ?, /, #, and .
       ':)?'
       '(?://'
         '(?:([^/?#]*)@)?'               // userInfo
-        '([\\w\\d\\-\\u0100-\\uffff.%]*)'
+        '(?:'
+          '([\\w\\d\\-\\u0100-\\uffff.%]*)|'

[floitsch, 2013/05/01 08:53:30]

I would prefer raw strings and not \\
Move the "|" into a separate line.

[Anders Johnsen, 2013/05/01 09:26:14]

Done.

                                         // domain - restrict to letters,
                                         // digits, dashes, dots, percent
                                         // escapes, and unicode characters.
+          '\\[([A-Fa-f0-9:.]*)\\])'

[floitsch, 2013/05/01 08:53:30]

Shouldn't you restrict the number of possible digits and ":" parts?

[Anders Johnsen, 2013/05/01 09:26:14]

We don't really do validations like that atm, but I'll leave a TODO.

+                                        // IPv6 domain - restrict to hex,
+                                        // dot and colon.
         '(?::([0-9]+))?'                // port
       ')?'
-      '([^?#]+)?'                       // path
+      '([^?#\\[]+)?'                       // path
       '(?:\\?([^#]*))?'                 // query
       '(?:#(.*))?'                      // fragment
       '\$');
 
   static const _COMPONENT_SCHEME = 1;
   static const _COMPONENT_USER_INFO = 2;
   static const _COMPONENT_DOMAIN = 3;
-  static const _COMPONENT_PORT = 4;
-  static const _COMPONENT_PATH = 5;
-  static const _COMPONENT_QUERY_DATA = 6;
-  static const _COMPONENT_FRAGMENT = 7;
+  static const _COMPONENT_DOMAIN_IPV6 = 4;
+  static const _COMPONENT_PORT = 5;
+  static const _COMPONENT_PATH = 6;
+  static const _COMPONENT_QUERY_DATA = 7;
+  static const _COMPONENT_FRAGMENT = 8;
 
   /**
    * Returns `true` if the URI is absolute.
    */
   bool get isAbsolute {
     if ("" == scheme) return false;
     if ("" != fragment) return false;
     return true;
 
     /* absolute-URI  = scheme ":" hier-part [ "?" query ]
@@ skipping 114 matching lines @@
     }
     return sb.toString();
   }
 
   String toString() {
     StringBuffer sb = new StringBuffer();
     _addIfNonEmpty(sb, scheme, scheme, ':');
     if (hasAuthority || (scheme == "file")) {
       sb.write("//");
       _addIfNonEmpty(sb, userInfo, userInfo, "@");
-      sb.write(domain == null ? "null" : domain);
+      sb.write(domain == null ? "null" :
+          domain.contains(':') ? '[$domain]' : domain);
       if (port != 0) {
         sb.write(":");
         sb.write(port.toString());
       }
     }
     sb.write(path == null ? "null" : path);
     _addIfNonEmpty(sb, query, "?", query);
     _addIfNonEmpty(sb, fragment, "#", fragment);
     return sb.toString();
   }
@@ skipping 20 matching lines @@
   }
 
   static void _addIfNonEmpty(StringBuffer sb, String test,
                              String first, String second) {
     if ("" != test) {
       sb.write(first == null ? "null" : first);
       sb.write(second == null ? "null" : second);
     }
   }
 }