Add support for @CookieParameter #216

Closed
cescoffier opened this Issue May 11, 2014 · 1 comment

Comments

Projects
None yet
1 participant
@cescoffier
Member

cescoffier commented May 11, 2014

Issue related to #204.

The cookie's name is passed as parameter (special values are provided to access the flash and session cookies). The value is converted to the parameter's type.

@cescoffier cescoffier added this to the 0.6 milestone May 11, 2014

@cescoffier cescoffier changed the title from Add support for @CookieParam to Add support for @CookieParameter May 12, 2014

cescoffier added a commit that referenced this issue May 15, 2014

Fix #215
Provide the @HttpParameter support. It also discards #216 as Cookie can be injected using @HttpParameter too.

Signed-off-by: Clement Escoffier <clement.escoffier@gmail.com>

@cescoffier cescoffier added the invalid label May 15, 2014

@cescoffier

This comment has been minimized.

Show comment
Hide comment
@cescoffier

cescoffier May 15, 2014

Member

This issue wont be fixed as the @HttpParameter annotation also support cookie:

@HttpParameter("hello") Cookie cookie : injects the 'hello' cookie
@HttpParameter SessionCookie cookie : injects the session cookie
@HttpParameter FlashCookie cookie : injects the flash cookie
Member

cescoffier commented May 15, 2014

This issue wont be fixed as the @HttpParameter annotation also support cookie:

@HttpParameter("hello") Cookie cookie : injects the 'hello' cookie
@HttpParameter SessionCookie cookie : injects the session cookie
@HttpParameter FlashCookie cookie : injects the flash cookie

@cescoffier cescoffier closed this May 15, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment