You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by "Ben Weidig (Jira)" <ji...@apache.org> on 2022/06/19 14:57:00 UTC
[jira] [Created] (TAP5-2736) CookieBuilder maxAge should support java.time.Duration
Ben Weidig created TAP5-2736:
--------------------------------
Summary: CookieBuilder maxAge should support java.time.Duration
Key: TAP5-2736
URL: https://issues.apache.org/jira/browse/TAP5-2736
Project: Tapestry 5
Issue Type: Improvement
Components: tapestry-core
Affects Versions: 5.8.2
Reporter: Ben Weidig
The CookieBuilder maxAge is based on seconds as int, but the contributed default value is using the intermediate type org.apache.tapestry5.ioc.util.TimeInterval which allows for defining an interval as a String.
This makes setting the default quite readable, but CookieBuilder#setMaxAge itself accepts an int and doesn't mention that it's supposed to be seconds.
I propose the following changes:
* Clarify documentation of CookieBuilder#setMaxAge so it's obvious that it requires seconds
* Add #setMaxAge(java.time.Duration)
* Add TypeCoercion Duration -> TimeInterval, so the default value could be contributed as String or Duration, to ensure backward-compatibility.
* Fix the use of the symbol key in the CookieBuilder constructor to use SymbolConstants.COOKIE_MAX_AGE instead of the value
--
This message was sent by Atlassian Jira
(v8.20.7#820007)