The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #1560

Bug in mailalias

Added by R.I. Pienaar almost 6 years ago. Updated over 5 years ago.

Status:ClosedStart date:09/05/2008
Priority:NormalDue date:
Assignee:Paul Lathrop% Done:

90%

Category:mailalias
Target version:0.25.0
Affected Puppet version:0.24.7 Branch:
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

I’m trying to use mailalias to manage a bunch of RT queue aliases, the typical /etc/aliases line would look like this:

test: “|/path/to/rt-mailgate —queue ‘test’ —action correspond —url http://my.com/”

If I use the mailalias type to do this using the code below:

mailalias{“test”:

    target          => "/tmp/aliases",
    recipient       => "|/path/to/rt-mailgate --queue 'test' --action correspond --url http://my.com/"

}

mailalias{“anothertest”:

    target          => "/tmp/aliases",
    recipient       => "|/path/to/rt-mailgate --queue 'another test' --action correspond --url http://my.com/"

}

The resulting /tmp/aliases file will be:

anothertest: “|/path/to/rt-mailgate —queue ‘another test’ —action correspond —url http //my.com/” test: “|/path/to/rt-mailgate —queue ‘test’ —action correspond —url http://my.com/”

Notice that the url in the first one is lacking the “:”, in my tests creating many aliases only one alias entry per file would be correct all the others would be wrong.

History

#1 Updated by James Turnbull almost 6 years ago

  • Category set to mailalias
  • Status changed from Unreviewed to Accepted
  • Assignee set to Puppet Community
  • Target version set to 4

No idea what’s happening here from looking at the code. Anyone have any ideas?

#2 Updated by Luke Kanies almost 6 years ago

Looks like the provider is just splitting on ‘:’, which doesn’t work since there really is only one colon that matters in the line.

#3 Updated by Paul Lathrop almost 6 years ago

  • Assignee changed from Puppet Community to Paul Lathrop

#4 Updated by Paul Lathrop over 5 years ago

  • Target version changed from 4 to 0.25.0
  • % Done changed from 0 to 10
  • 3 changed from Unknown to Easy

Just wanted to let folks know I’m actually working on this :–)

#5 Updated by Paul Lathrop over 5 years ago

  • Status changed from Accepted to Ready For Checkin
  • Affected Puppet version changed from 0.24.4 to 0.24.7

Patch submitted to dev list for review. Also available on GitHub: http://github.com/plathrop/puppet/tree/ticket/0.24.x/1560

#6 Updated by Paul Lathrop over 5 years ago

  • % Done changed from 10 to 90

#7 Updated by James Turnbull over 5 years ago

  • Status changed from Ready For Checkin to Closed

Merged in commit:234a035b97287d8910ca00a58b70a06775818693 in branch master.

Also available in: Atom PDF