undefined variable en in auth.py line 279

[darth Dingleberry]

Hi, First off thanks for keeping this critical plugin moving along ;-) Now my little typo find: I think this should be "e" not "en" as that makes it put out a proper error message.

Line is the raise statement:

try:
    self._ldap.simple_bind_s(self.dir_binddn, self.dir_bindpw)
except ldap.LDAPError, e:
    self.log.info('bind_ad: binding failed. %s ' % (e))
    raise TracError('cannot bind to %s: %s' % (self.dir_uri , en))

Also I would like to work on getting memberOf working as we need that functionality in order to use this tool. How would I go about making that level of contribution? I have another python tool that does most of the work already in place. Lastly we have to force referrals off to make this work, currently that means locally forked code. I would like to get that worked in as well to trunk.

Thanks, Eddie J

ps apologies for the email mixed with a bug report, not sure how to get into the "loop"

[lkraav]

Yeah, confirming the typo part. It's a rather low cost patch, I wonder if sandinak could push it to the repo with some urgency? Is he still interested in the plugin?

[Ryan J Ollos]

(In [12713]) Fixes #10932: Fixed typo in passing exception object to TracError.

[Ryan J Ollos]

Replying to lkraav:

Is he still interested in the plugin?

To speak to darth_dingleberry's earlier question, if sandinak isn't around any longer, you can request to take over maintenance of the plugin; see AdoptingHacks. However, I think sandinak is around, so hopefully we hear back shortly.

[darth Dingleberry]

rjollos thanks for the quick fix. I will wait for sandinak to reply and keep working on my local copy. I would like to earn commit privs after putting up a few patches.

thanks Eddie

[branson]

so I am around .. just havn't had the bw to contribute to this recently .. I'm building a trac-1.0.1 release today for other stuff .. and will test the plugin and go thru the open issues. Sorry for the delay.. been busy as crap with 10000 other things.

[lkraav]

I have been running this on top of current 1.1.1-dev trunk, it's working without issues.

But there's definitely stuff to do and improve on. Trying to find that patching time somewhere myself.. Stay tuned for some additional tickets :)

[darth Dingleberry]

lkraav would you like to PM about what your thoughts are on expanding this plugin?

My first and foremost is gettting memberOf working for us so we can use this natively and get rid of all the Apache baggage.

Also a few others that would be nice to try and get handled if we can together work up some bandwidth. And of course whatever sandinak has cooking.

Thanks dD

[lkraav]

I think it would make sense to kick off a public discussion thread in trac-dev mailing list instead, or just straight file feature tickets right here.