Bug 246218 - Is public static final String AUTH_COOKIE_NAME needed?
Is public static final String AUTH_COOKIE_NAME needed?
Product: Red Hat Satellite 5
Classification: Red Hat
Component: Other (Show other bugs)
All Linux
low Severity low
: ---
: ---
Assigned To: Jan Pazdziora
Preethi Thomas
Depends On:
Blocks: 248627
  Show dependency treegraph
Reported: 2007-06-29 06:27 EDT by Jan Pazdziora
Modified: 2008-04-02 20:18 EDT (History)
0 users

See Also:
Fixed In Version: sat510
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-04-02 20:18:39 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Jan Pazdziora 2007-06-29 06:27:25 EDT
Description of problem:

I was investigating the request from


By grepping the SVN for 'rh_auth_token', I found it used in

java/code/src/com/redhat/rhn/frontend/struts/RhnHelper.java:    public static
final String AUTH_COOKIE_NAME = "rh_auth_token";
java/code/src/com/redhat/rhn/frontend/struts/RequestContext.java:    public
static final String AUTH_COOKIE_NAME = "rh_auth_token";

But AUTH_COOKIE_NAME is not to be found anywhere else. Should we be able to
remove it from the source code? The problem reported in that mailing-list post
is no longer present on Satellite 5.0.0, so I assume we've removed the use of
these but not the declaration.

Version-Release number of selected component (if applicable):

SVN trunk as of today (revision 117910).

How reproducible:

Looked at the source code.

Steps to Reproduce:
1. Grepped the source AUTH_COOKIE_NAME.
Actual results:

In the Java code, it is only found in frontend/struts/RhnHelper.java and
frontend/struts/RequestContext.java as declarations. It does not seem to be used

Expected results:

Additional info:
Comment 1 Jan Pazdziora 2007-10-09 05:43:13 EDT
Reassigning to myself.

This got all three acks so I assume we want this change to happen.

Committed to SVN, trunk, revision 132727.
Comment 2 Jan Pazdziora 2007-10-09 14:49:51 EDT
Moving ON_QA, with:

Satellite 5.1.0-14 Content now available in qa Channels.

Satellite 5.1.0-14 ISOs are now available as
rhn-satellite-5.1.0-14-redhat-linux-as-* @:

Comment 3 Preethi Thomas 2007-10-29 11:42:47 EDT
Fetch the rhn-java-sat.src.rpm, install it, go to the SPECS directory, run rpm
-bp ....spec, go to SOURCES directory, grep -r the source and see if that string
is gone.
adelton Preethi_afk: ^^^ would approximately be the way I'd do it.
adelton Preethi_afk: s/rpm -bp/rpmbuild -bp/
Comment 4 Brandon Perkins 2008-04-02 20:18:39 EDT
5.1 Sat GA so Closed for Current Release.

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