Issue #21102 has been updated by Alex Dziena. Status changed from Unreviewed to Needs More Information Assignee set to Mark Zeren
Mark, can you specify the affected Puppet version and open a pull request for your fix? ---------------------------------------- Bug #21102: opaque AuthStore::Declaration matching is broken for names with periods and upper case. https://projects.puppetlabs.com/issues/21102#change-94536 * Author: Mark Zeren * Status: Needs More Information * Priority: Normal * Assignee: Mark Zeren * Category: network * Target version: * Affected Puppet version: * Keywords: * Branch: ---------------------------------------- In AuthStore::Declaration, parse sets @pattern to [value], while matchname? compares pattern to munge_name(value). These two expressions will never match if the value contains uppercase characters or '.', however the regex for :opaque allows both . and uppercase. I have a draft fix with matching tests available here: https://github.com/mzeren-vmw/puppet/commits/fix_opaque_decl Note that this is related to / resolves one of the issues discussed in #21101 -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
