Details
Bug
Major
Description
This bug is related to ICE-893. When trying to get HttpServletRequest header
information from a call to
FacesContext.getCurrentInstance().getExternalContext().getRequest() the header
information was always null. After tracing the problem from the stack trace
the method copyServletRequestData() was copying the header information correctly
then setting it to null for no apparent reason.
As it turns out the logging api will only be invoked on action or action
listeners so it will never acually access the header data via the
ServletEnvironmentRequest object because it is only used by the
PersistentFacesServlet .
information from a call to
FacesContext.getCurrentInstance().getExternalContext().getRequest() the header
information was always null. After tracing the problem from the stack trace
the method copyServletRequestData() was copying the header information correctly
then setting it to null for no apparent reason.
As it turns out the logging api will only be invoked on action or action
listeners so it will never acually access the header data via the
ServletEnvironmentRequest object because it is only used by the
PersistentFacesServlet .
Activity
| Field | Original Value | New Value |
|---|---|---|
| issue.field.bugzillaimportkey | 913 | 12163 |
| Affects Version/s | 1.5 [ 10027 ] | |
| Affects Version/s | unspecified [ 10021 ] |
Re-assigning to Patrick to use the desired feature in the application, re-test with the current build, and then assign to Mircea if a fix is necessary.
Show
Ted Goddard
added a comment - Re-assigning to Patrick to use the desired feature in the application, re-test with the current build, and then assign to Mircea if a fix is necessary.
| Assignee | Ted Goddard [ ted.goddard ] | Patrick Corless [ patrick.corless ] |
| Fix Version/s | 1.6DR#4 [ 10060 ] | |
| Fix Version/s | 1.6 [ 10031 ] |
Closing bug.
As of 1.6 dr3 this bug has been addressed. The refactoring of the core that has taken place between 1.5 and 1.6 has fixed the problem. The only thing to note is that when a call to getHeader(String param) is make the param String must be lower case, for example "content-type" instead of the normal convention "Content-Type".
I'll recreate a new bug to address cases sensitivity issue with the header name values pairs.
Show
Patrick Corless
added a comment - Closing bug.
As of 1.6 dr3 this bug has been addressed. The refactoring of the core that has taken place between 1.5 and 1.6 has fixed the problem. The only thing to note is that when a call to getHeader(String param) is make the param String must be lower case, for example "content-type" instead of the normal convention "Content-Type".
I'll recreate a new bug to address cases sensitivity issue with the header name values pairs.
Marking as fixed.
Show
Patrick Corless
added a comment - Marking as fixed.
| Status | Open [ 1 ] | Closed [ 6 ] |
| Resolution | Fixed [ 1 ] |
icefaces.org can use a modified post-release build. Recommend we address later.