http://bugzilla.spamassassin.org/show_bug.cgi?id=3826

[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[EMAIL PROTECTED]



------- Additional Comments From [EMAIL PROTECTED]  2004-12-13 22:14 -------
I can reproduce this under 3.0.1 by starting spamd with -m1 (along with -p785 -D
-L) and then running a message through with spamc while having "version_tag
whatever" in the current user's user_prefs file.  If I then remove the
version_tag line from the same user's user_prefs file, and run the message
through with spamc again, the version tag still shows up in the output.

The relevant code (from Conf.pm):

  push (@cmds, {
    setting => 'version_tag',
    code => sub {
      my ($self, $key, $value, $line) = @_;
      my $tag = lc($value);
      $tag =~ tr/a-z0-9./_/c;
      foreach (@Mail::SpamAssassin::EXTRA_VERSION) {
        if($_ eq $tag) { $tag = undef; last; }
      }
      push(@Mail::SpamAssassin::EXTRA_VERSION, $tag) if($tag);
    }
  });


Cause:

The version_tag is just pushed on the array in M::SA, not stored in the {conf} 
hash.


Solution:

Adding "is_admin => 1" to the hash (above) would (obviously) stop this from
happening.  Is there any reason a user would have a real need to use the
version_tag, other than if allow_user_rules is enabled (in which case this
should probably be a 'part of' that)?


In any case, the remaining bug is only cosmetic (user's could swear at each
other via their user_prefs files) and isn't related to the copy_config() issues.


Daryl




Daryl



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to