Uploaded image for project: 'Harmony'
  1. Harmony
  2. HARMONY-1127

[classlib][net] compatibility: Harmony constructor URI(String * 5) throws URISyntaxExceptionwhile RI does not

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Labels:
      None

      Description

      The Harmony method java.net.URI(String scheme, String authority, String path, String query, String fragment) throws URISyntaxException for incorrect ipv6 address while RI silently skip it.
      Note, the URI unit test for this constructor failed on RI and passed on Harmony.

      ====================== test.java =======================
      import java.net.*;

      public class test {
      public static void main (String[] args) throws Exception

      { System.out.println("res = " + new URI("ftp", "[www.abc.com]", "/dir", "query", "frag")); }


      }
      ====================================================

      Output:
      C:\tmp\tmp17>C:\jdk1.5.0_06\bin\java.exe -cp . -showversion test
      java version "1.5.0_06"
      Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_06-b05)
      Java HotSpot(TM) Client VM (build 1.5.0_06-b05, mixed mode)

      res = ftp:///dir?query#frag

      C:\tmp\tmp17>C:\harmony\classlib1.5\deploy\jdk\jre\bin\java.exe -cp . -showversion test
      java version 1.5 (subset)

      (c) Copyright 1991, 2006 The Apache Software Foundation or its licensors, as applicable.
      Exception in thread "main" java.net.URISyntaxException: Malformed ipv6 address: [www.abc.com]
      at java.net.URI$Helper.isValidHost(URI.java:467)
      at java.net.URI$Helper.parseAuthority(URI.java:431)
      at java.net.URI$Helper.parseURI(URI.java:31)
      at java.net.URI$Helper.access$100(URI.java:22)
      at java.net.URI.<init>(URI.java:199)
      at test.main(test.java:5)

        Activity

        Hide
        cap Alexey Petrenko added a comment -

        But Harmony seems correct here.

        Show
        cap Alexey Petrenko added a comment - But Harmony seems correct here.
        Hide
        mmarkov Mikhail Markov added a comment -

        I agree with Alexey: RI just swallows the host information silently and does not store it internally (as could be seen by RI's output) which is incorrect anyway, imho.

        I suggest moving this bug to "Non-bug differences from RI" category and close it.

        Show
        mmarkov Mikhail Markov added a comment - I agree with Alexey: RI just swallows the host information silently and does not store it internally (as could be seen by RI's output) which is incorrect anyway, imho. I suggest moving this bug to "Non-bug differences from RI" category and close it.
        Hide
        tellison Tim Ellison added a comment -

        Changing resolution.

        Show
        tellison Tim Ellison added a comment - Changing resolution.

          People

          • Assignee:
            tellison Tim Ellison
            Reporter:
            vladimir Vladimir Ivanov
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development