ICEfaces
  1. ICEfaces
  2. ICE-2410

occasional NullPointerException with WebMC on GlassFish

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.7DR#2
    • Fix Version/s: 1.7.2
    • Component/s: None
    • Labels:
      None
    • Environment:
      ICEfaces, glassfish-v2-ur1-b08
    • Workaround Exists:
      Yes
    • Workaround Description:
      Hide
      + uri = URI.create("http://" + req.getServerName() + ":"
      + + req.getServerPort() + req.getRequestURI());

      Show
      + uri = URI.create("http://" + req.getServerName() + ":" + + req.getServerPort() + req.getRequestURI());

      Description


      GlassFish/Grizzly integration is now working well with the auctionMonitor, however occasional NullPointerExeptions were seen with WebMC. The following workaround and debugging code was applied for this. The NullPointerException is due to the getScheme() returning null on the request. A current version of GlassFish must be tested with WebMC to verify this.



      +++ src/com/icesoft/faces/webapp/http/servlet/ServletRequestResponse.java (working copy)
      @@ -45,7 +45,18 @@
                   public void servlet(Object request, Object response) {
                       HttpServletRequest req = (HttpServletRequest) request;
                       String query = req.getQueryString();
      - URI uri = URI.create(req.getRequestURL().toString());
      + URI uri = null;
      + try {
      + uri = URI.create(req.getRequestURL().toString());
      + } catch (Exception e) {
      + System.out.println("building our own URI " + e);
      + uri = URI.create("http://" + req.getServerName() + ":"
      + + req.getServerPort() + req.getRequestURI());
      + System.out.println("Constructing URI " + uri + " scheme " + req.getScheme());
      + }
      + }
                       requestURI = (query == null ? uri : URI.create(uri + "?" + query));
                   }

        Activity

        Ted Goddard created issue -
        Ted Goddard made changes -
        Field Original Value New Value
        Fix Version/s 1.7 [ 10080 ]
        Ted Goddard made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.7DR#3 [ 10112 ]
        Fix Version/s 1.7 [ 10080 ]
        Resolution Fixed [ 1 ]
        Ted Goddard made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Mandeep Hayher [ mandeep.hayher ]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Fix Version/s 1.7DR#3 [ 10112 ]
        Ken Fyten made changes -
        Assignee Mandeep Hayher [ mandeep.hayher ] Jack van Ooststroom [ jack.van.ooststroom ]
        Ken Fyten made changes -
        Fix Version/s 1.7 [ 10080 ]
        Assignee Jack van Ooststroom [ jack.van.ooststroom ] Greg Dick [ greg.dick ]
        Priority Major [ 3 ] Minor [ 4 ]
        Ken Fyten made changes -
        Assignee Greg Dick [ greg.dick ] Ted Goddard [ ted.goddard ]
        Ted Goddard made changes -
        Fix Version/s 1.7.2 [ 10130 ]
        Ted Goddard made changes -
        Assignee Ted Goddard [ ted.goddard ] Jack Van Ooststroom [ jack.van.ooststroom ]
        Ken Fyten made changes -
        Fix Version/s 1.8DR#2 [ 10142 ]
        Fix Version/s 1.7.2 [ 10130 ]
        Ken Fyten made changes -
        Assignee Jack Van Ooststroom [ jack.van.ooststroom ] Greg Dick [ greg.dick ]
        Deryk Sinotte made changes -
        Assignee Priority P3
        Ken Fyten made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Fix Version/s 1.7.2 [ 10130 ]
        Fix Version/s 1.8DR#2 [ 10142 ]
        Assignee Priority P3
        Resolution Fixed [ 1 ]
        Assignee Greg Dick [ greg.dick ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Unassigned
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: