Menu

#11 Rewrite cookie handling to use JDK facilities

accepted
EJP
None
5
2022-02-21
2021-09-20
EJP
No

JDK supports cookie handling natively via the CookieManager. Use this and discard present code. Remove the per-site use-cookie configuration and replace with standard cookie policies: all, none, or originating server only.

This will introduce a minor binary incompatibility, as the present Cookie class will cease to be used, and java.net.HttpCookie will appear instead in a couple of places in the Site interface and throughout the Storage subsystem. There will then be no real reason to have CookieDAO and friends at all. Alternatively a JDBC CookieStore implementation could be created.

Discussion

  • EJP

    EJP - 2021-10-06
    • assigned_to: EJP
     
  • EJP

    EJP - 2021-10-06
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,3 +1,3 @@
     JDK supports cookie handling natively via the CookieManager. Use this and discard present code. Remove the per-site use-cookie configuration and replace with standard cookie policies: all, none, or originating server only.
    
    -This will introduce a minor binary incompatibilty, as the present Cookie class will cease to be used, and java.net.HttpCookie will appear instead in a couple of places in the Site interface and throughout the Storage subsystem. There will then be no real reason to have CookieDAO and friends at all. Alternatively a JDBC CookieStore implementation could be created.
    +This will introduce a minor binary incompatibility, as the present Cookie class will cease to be used, and java.net.HttpCookie will appear instead in a couple of places in the Site interface and throughout the Storage subsystem. There will then be no real reason to have CookieDAO and friends at all. Alternatively a JDBC CookieStore implementation could be created.
    
     
  • EJP

    EJP - 2022-02-21
    • status: open --> accepted
     

Log in to post a comment.