problem with conditionalhttp

I did some poking around in CVS and it looks like this line in 
conditionalhttp wasn't in any other branch, but got checked in with Ted's 
merge from HEAD to PYBLOSXOM_1_2:

Line 49:

if config.has_key('comment_dir') and not "rss" in 
http.get('REQUEST_URI','') :

That "rss" in http.get('REQUEST_URI', '') part hasn't been checked in 
before anywhere and it looks like something Ted did locally on his blog, 
maybe?

Anyhow, that isn't valid Python 2.2.x code.

So, my questions are two-fold.  First, is that analysis right?  Second, if 
the analysis is right, is it ok if I back that line out and release 
another contributed plugins pack?

/will


---------- Forwarded message ----------
Date: Sat, 7 May 2005 16:28:48 -0400
From: Joseph Reagle <[email protected]>
To: pyblosxom-users-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org
Cc: will guaraldi <[email protected]>
Subject: Re: [pyblosxom-users] contrib 1.2.1 for PyBlosxom 1.2 release

On Saturday 07 May 2005 14:11, will guaraldi wrote:
> You can get the contributed plugin pack at:
>
>    https://sourceforge.net/project/showfiles.php?group_id=67445

I'm not sure what this problem is:

http://reagle.org/joseph/pyblosxom2.cgi
-->

<snip really long stacktrace thing>

  /home/reagle/data/2web/pyblosxom-1.2/contrib-1.2.1/plugins/conditionalhttp.py
in cb_prepare(args={'request': Request})

TypeError: 'in ' requires character as left operand
       __doc__ = 'Inappropriate argument type.'
       __getitem__ = <bound method TypeError.__getitem__ of
<exceptions.TypeError instance>>
       __init__ = <bound method TypeError.__init__ of <exceptions.TypeError
instance>>
       __module__ = 'exceptions'
       __str__ = <bound method TypeError.__str__ of <exceptions.TypeError
instance>>
       args = ("'in <string>' requires character as left operand",)


-------------------------------------------------------
This SF.Net email is sponsored by: NEC IT Guy Games.
Get your fingers limbered up and give it your best shot. 4 great events, 4
opportunities to win big! Highest score wins.NEC IT Guy Games. Play to
win an NEC 61 plasma display. Visit http://www.necitguy.com/?r=20