Bug 144409 - NumberFormatException thrown if no web.allow_pxt_personalities is defined.
NumberFormatException thrown if no web.allow_pxt_personalities is defined.
Status: CLOSED CURRENTRELEASE
Product: Red Hat Network
Classification: Red Hat
Component: RHN/R&D (Show other bugs)
RHN Devel
All Linux
medium Severity medium
: ---
: ---
Assigned To: Ryan Bloom
Red Hat Satellite QA List
:
Depends On:
Blocks: rhnMilestone1
  Show dependency treegraph
 
Reported: 2005-01-06 15:02 EST by Jesus M. Rodriguez
Modified: 2007-04-18 13:17 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2005-01-20 14:26:28 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)

  None (edit)
Description Jesus M. Rodriguez 2005-01-06 15:02:06 EST
An exception is thrown if web.allow_pxt_personalities is not
present or of the wrong format in the configuration files.
Config.getInt() should throw a NumberFormatException.
And the call of that should handle such an error.

Files affected:
RhnHelper.java: getSessionCookieName, line 408
Config.java: getInt, line 165

Exception:
java.lang.NumberFormatException: null
	java.lang.Integer.parseInt(Integer.java(Compiled Code))
	java.lang.Integer.parseInt(Integer.java(Compiled Code))
	com.redhat.rhn.common.conf.Config.getInt(Config.java:165)
	com.redhat.rhn.frontend.struts.RhnHelper.getSessionCookieName(RhnHelper.java:408)
	com.redhat.rhn.frontend.struts.RhnHelper.findUserSession(RhnHelper.java:134)
	com.redhat.rhn.frontend.servlets.AuthFilter.doFilter(AuthFilter.java:65)
	com.opensymphony.module.sitemesh.filter.PageFilter.parsePage(PageFilter.java:142)
	com.opensymphony.module.sitemesh.filter.PageFilter.doFilter(PageFilter.java:58)
	com.redhat.rhn.frontend.servlets.EnvironmentFilter.doFilter(EnvironmentFilter.java:73)
Comment 1 Ryan Bloom 2005-01-20 14:26:08 EST
Asking for a non-existant config variable shouldn't throw an exception
at all.  It should return a sane default, in this case 0.  The code
now does this.

Note You need to log in before you can comment on or make changes to this bug.