]> review.fuel-infra Code Review - puppet-modules/puppetlabs-firewall.git/commit
Lookup username from uuid
authorDan Bode <bodepd@gmail.com>
Thu, 8 Jan 2015 17:06:10 +0000 (18:06 +0100)
committerJonathan Tripathy <jonathan.tripathy@puppetlabs.com>
Wed, 21 Jan 2015 07:40:32 +0000 (23:40 -0800)
commit86620cff165fbffe623e0c8c7bde1204ee76bf30
tree71b888b8c3033b5b7f52506d5ddbc1b0d297281f
parentb2eae7356785bdcbf26fdafd0874f4843d01baff
Lookup username from uuid

When using the uid feature of the firewall module,
it did not work with string based usernames as
documented.

The uid propery always synchronized with a message of
<number> does not match <username>.

This code overrides the uid getter method to perform
a check of both the data from the property hash as well
as using that data (assuming it is a uid) to resolve the
username.

While this patch is pretty simple, I have only tested it
on Ubuntu 14.04. I am not sure if it could be problematic
with other versions.

I have not yet written tests b/c I wanted to submit
my proposed fix for discussion while I get those
written.
lib/puppet/type/firewall.rb