Issue #21102 has been updated by Mark Zeren.
I would suggest closing this bug for now. I was able to accomplish my goal (Kerberos support) in a different way that doesn't touch AuthStore. In the end I found many design issues and bugs in the AuthStore. Resolving many of these would potentially break backward compatibility. My assessment was that AuthStore should get an overhaul on a major release boundary. ---------------------------------------- Bug #21102: opaque AuthStore::Declaration matching is broken for names with periods and upper case. https://projects.puppetlabs.com/issues/21102#change-96212 * 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.
