Re: [webmin-l] LDAP Users Module Feature Request - IMAP Quota Field

[Murray T. <mtr...@ce...>]

On Sun, 2006-07-23 at 22:20 -0700, Jamie Cameron wrote:
> On 23/Jul/2006 18:55 Murray Trainer wrote ..
> > On Fri, 2006-07-21 at 12:37 -0700, Jamie Cameron wrote:
> > > On 19/Jul/2006 19:02 Murray Trainer wrote ..
> > > > Hi Jamie,
> > > > 
> > > > Thanks for all your modifications to this module to date.  I haven't
> > > > asked for anything for a while so here goes :-)  The module currently
> > > > has a hard coded IMAP quota value in its config which has been fine
> > > > until now.  We now need to create different classes of users with
> > > > different IMAP quotas.  We use the Cyrus IMAP module for maintaining
> > > > user mailboxes but want to keep the user creation process to a single
> > > > step.  An IMAP quota field in the user creation/edit screen that can
> > > > override the module's default IMAP quota would be really helpful. 
> > > > 
> > > > It appears you use the NET::IMAP perl module in ldap-useradmin-lib.pl.
> > > > The code you use to set the user's quota in save_user.cgi is:
> > > > 
> > > >        if ($config{'quota'}) {
> > > >                 # Set his IMAP quota
> > > >                 $rv = $imap->setquota("user.".$user,
> > > >                                       "STORAGE", $config{'quota'});
> > > >                 $rv->{'Status'} eq 'ok' ||
> > > >                         &imap_error($text{'usave_eiquota'}, $rv);
> > > >                 }
> > > > 
> > > > I would assume you could use the getquota function described below
> > to
> > > > get the current quota on the mailbox.  
> > > > 
> > > > http://search.cpan.org/~kjohnson/NetxAP-0.02/Net/IMAP.pm#getquota_%
> > > > 24quotaroot
> > > > 
> > > > Hope you're not too busy to have a look at this one if you think it's
> > a
> > > > reasonable idea.
> > > 
> > > That is quite a good idea, and I have just added it to the module. However
> > > I don't actually run a Cyrus IMAP server myself, so I can't fully test
> > it!
> > > But you can, by downloading the module from :
> > > 
> > > http://www.webmin.com/updates/ldap-useradmin-1.290-4.wbm.gz
> > > 
> > > Let me know how it goes.
> > > 
> > >  - Jamie
> > 
> > I upgraded the ldap-useradmin from 1.273 to the above. It brings up the
> > add user form with the default quota OK but I get the error below when
> > saving the user.  Not sure where that problem slipped in - doesn't sound
> > related to your change. 
> > 
> > Failed to save user : Failed to add user to LDAP database : attribute
> > 'givenName' provided more than once
> 
> I did also fix a bug related to that field in this new version, so that
> whatever you enter into the 'First name' field is set as the givenName
> LDAP attribute.
> 
> Do you have a custom setting on the Module Config page for the givenName
> attribute as well? If so, it should probably be removed..
> 
>  - Jamie

Hi Jamie,

I just had another look at upgrading the LDAP Users module to your test
version but I still get the error "attribute 'givenName' provided more
than once" above.  I don't have any custom settings to do with
givenName.  Of course I have the setting "Show fields for given name and
surname" on.  Somehow Webmin is trying to save that attribute twice?
Could you maybe look at the code that was fixed and maybe see how it
could cause my problem?

Thanks

Murray