Details

    • Issuezilla Id:
      733
    • Status Whiteboard:
      Hide

      cat2 javadoc size_small importance_small cat3

      Show
      cat2 javadoc size_small importance_small cat3

      Description

      From https://javaserverfaces.dev.java.net/issues/show_bug.cgi?id=1524:

      Both ServletRequest and PortletRequest have a "public boolean isSecure();"
      method, but no such method is provided on ExternalContext. Since they are the
      same, ExternalContext should be providing the method so that developers do not
      need to cast the request object.

        Activity

        Hide
        Ed Burns added a comment -

        cat2

        Show
        Ed Burns added a comment - cat2
        Hide
        Ed Burns added a comment -

        javadoc

        Show
        Ed Burns added a comment - javadoc
        Hide
        Ed Burns added a comment -

        These are targeted at 2.1.

        Show
        Ed Burns added a comment - These are targeted at 2.1.
        Hide
        sheetalv added a comment -

        triage

        Show
        sheetalv added a comment - triage
        Hide
        Ed Burns added a comment -

        edburns

        Show
        Ed Burns added a comment - edburns
        Hide
        Ed Burns added a comment -

        Change target milestone.

        Show
        Ed Burns added a comment - Change target milestone.
        Hide
        Ed Burns added a comment -

        Security related, 2.1

        Show
        Ed Burns added a comment - Security related, 2.1
        Hide
        sheetalv added a comment -

        changing owner to sheetalv

        Show
        sheetalv added a comment - changing owner to sheetalv
        Hide
        Ed Burns added a comment -

        cat3

        Show
        Ed Burns added a comment - cat3
        Hide
        sheetalv added a comment -

        issue has been fixed in r8512 - trunk.

        Log message as follows :
        -----------------------------------
        Changes for spec issue 733 fix :

        Mojarra :
        M jsf-api/src/main/java/javax/faces/context/ExternalContext.java
        M jsf-ri/test/com/sun/faces/context/TestExternalContextImpl.java
        M jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java
        M lib/jsf-extensions-test-time.jar

        jsf-extensions :
        M test-time/src/main/java/com/sun/faces/mock/MockExternalContext.java

        Added the isSecure() method to the API, RI and test.

        Reviewed by Ed Burns

        Show
        sheetalv added a comment - issue has been fixed in r8512 - trunk. Log message as follows : ----------------------------------- Changes for spec issue 733 fix : Mojarra : M jsf-api/src/main/java/javax/faces/context/ExternalContext.java M jsf-ri/test/com/sun/faces/context/TestExternalContextImpl.java M jsf-ri/src/main/java/com/sun/faces/context/ExternalContextImpl.java M lib/jsf-extensions-test-time.jar jsf-extensions : M test-time/src/main/java/com/sun/faces/mock/MockExternalContext.java Added the isSecure() method to the API, RI and test. Reviewed by Ed Burns
        Hide
        Ed Burns added a comment -

        add changelog_2_1 keyword

        Show
        Ed Burns added a comment - add changelog_2_1 keyword
        Hide
        Ed Burns added a comment -

        Update status for JCP ChangeLog.

        Show
        Ed Burns added a comment - Update status for JCP ChangeLog.
        Hide
        Ed Burns added a comment -

        Ensure changelog_2_1 is in keywords list

        Show
        Ed Burns added a comment - Ensure changelog_2_1 is in keywords list
        Hide
        Ed Burns added a comment -

        2.2

        Show
        Ed Burns added a comment - 2.2
        Hide
        Manfred Riem added a comment -

        Closing resolved issue out

        Show
        Manfred Riem added a comment - Closing resolved issue out

          People

          • Assignee:
            sheetalv
            Reporter:
            Jason Lee
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: