[TAPESTRY-1796] Not all writeCookieValue functions honor the tapestry.default-cookie-max-age symbol

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.0.14
• Fix Version/s: 5.0.15
• Component/s: Core Components
• Assignee: Howard Lewis Ship
• Reporter: Ned Jackson Lovely
• Created: 01/Oct/07 21:06
• Updated: 08/Sep/08 17:56
• Resolved: 08/Sep/08 17:56

Description

In org.apache.tapestry.internal.services.CookiesImpl, the functions

public void writeCookieValue(String name, String value, String path)
public void writeDomainCookieValue(String name, String value, String domain)
public void writeCookieValue(String name, String value, String path, String domain)

all need the line

cookie.setMaxAge(_defaultMaxAge);

...otherwise, they just set transient cookies, which doesn't appear to be the intended behavior, and is certainly not consistant behavior.