{"action":"created","issue":{"url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/issues/3","labels_url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/issues/3/comments","events_url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/issues/3/events","html_url":"https://github.com/hercules-team/augeasproviders_puppet/issues/3","id":104116119,"number":3,"title":"Diff backwards and Changes not applied","user":{"login":"acidprime","id":1173142,"avatar_url":"https://avatars.githubusercontent.com/u/1173142?v=3","gravatar_id":"","url":"https://api.github.com/users/acidprime","html_url":"https://github.com/acidprime","followers_url":"https://api.github.com/users/acidprime/followers","following_url":"https://api.github.com/users/acidprime/following{/other_user}","gists_url":"https://api.github.com/users/acidprime/gists{/gist_id}","starred_url":"https://api.github.com/users/acidprime/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/acidprime/subscriptions","organizations_url":"https://api.github.com/users/acidprime/orgs","repos_url":"https://api.github.com/users/acidprime/repos","events_url":"https://api.github.com/users/acidprime/events{/privacy}","received_events_url":"https://api.github.com/users/acidprime/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-08-31T19:14:52Z","updated_at":"2015-08-31T21:46:36Z","closed_at":null,"body":"Given the following auth.conf\r\n\r\n```\r\npath ~ ^/catalog/([^/]+)$\r\nmethod find\r\nauth yes\r\nallow $1\r\n\r\npath ~ ^/node/([^/]+)$\r\nmethod find\r\nauth yes\r\nallow $1\r\n\r\npath  /certificate_revocation_list/ca\r\nmethod find\r\nauth yes\r\nallow *\r\n\r\npath ~ ^/report/([^/]+)$\r\nmethod save\r\nauth yes\r\nallow $1\r\n\r\npath  /file\r\nauth yes\r\nallow *\r\n\r\npath  /certificate/ca\r\nmethod find\r\nauth any\r\nallow *\r\n\r\npath  /certificate/\r\nmethod find\r\nauth any\r\nallow *\r\n\r\npath  /certificate_request\r\nmethod find, save\r\nauth any\r\nallow *\r\n\r\npath  /v2.0/environments\r\nmethod find\r\nauth yes\r\nallow *\r\n\r\npath  /facts\r\nmethod find, search\r\nauth any\r\nallow pe-internal-dashboard\r\n\r\npath  /resource_type\r\nmethod find, search\r\nauth yes\r\nallow pe-internal-dashboard, pe-internal-classifier\r\n\r\npath  /\r\nauth any\r\n```\r\n\r\nWith the following puppet code\r\n\r\n```puppet\r\n  puppet_auth { 'allow the diff server to retrieve any catalog':\r\n    ensure        => present,\r\n    path          => '^/catalog/([^/]+)$',\r\n    path_regex    => true,\r\n    authenticated => 'yes',\r\n    methods       => 'find',\r\n    allow         => ['$1', $diff_master],\r\n  }\r\n\r\n  puppet_auth { 'allow the diff server to query facts':\r\n    ensure        => present,\r\n    path          => '/facts',\r\n    authenticated => 'any',\r\n    path_regex    => false,\r\n    methods       => [ 'find', 'search'],\r\n    allow         => ['$1', $diff_master, 'pe-internal-dashboard'],\r\n  }\r\n```\r\n\r\nCauses the no changes and results in a rather strange diff\r\n\r\n```diff\r\nNotice: /Stage[main]/Catalog_diff/Puppet_auth[allow the diff server to retrieve any catalog]/allow: allow changed ['$1'] to '$1 compile-master-1.vm'\r\nNotice: /Stage[main]/Catalog_diff/Puppet_auth[allow the diff server to query facts]/allow: allow changed ['pe-internal-dashboard'] to '$1 compile-master-1.vm pe-internal-dashboard'\r\nNotice: /Stage[main]/Puppet_enterprise::Profile::Master::Auth_conf/File[/etc/puppetlabs/puppet/auth.conf]/content:\r\n--- /etc/puppetlabs/puppet/auth.conf\t2015-08-31 19:11:55.472823920 +0000\r\n+++ /tmp/puppet-file20150831-22170-12rzjfw\t2015-08-31 19:11:55.482828920 +0000\r\n@@ -1,7 +1,7 @@\r\n path ~ ^/catalog/([^/]+)$\r\n method find\r\n auth yes\r\n-allow $1, compile-master-1.vm\r\n+allow $1\r\n\r\n path ~ ^/node/([^/]+)$\r\n method find\r\n@@ -45,7 +45,7 @@\r\n path  /facts\r\n method find, search\r\n auth any\r\n-allow $1, compile-master-1.vm, pe-internal-dashboard\r\n+allow pe-internal-dashboard\r\n\r\n path  /resource_type\r\n method find, search\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/issues/comments/136510396","html_url":"https://github.com/hercules-team/augeasproviders_puppet/issues/3#issuecomment-136510396","issue_url":"https://api.github.com/repos/hercules-team/augeasproviders_puppet/issues/3","id":136510396,"user":{"login":"raphink","id":650430,"avatar_url":"https://avatars.githubusercontent.com/u/650430?v=3","gravatar_id":"","url":"https://api.github.com/users/raphink","html_url":"https://github.com/raphink","followers_url":"https://api.github.com/users/raphink/followers","following_url":"https://api.github.com/users/raphink/following{/other_user}","gists_url":"https://api.github.com/users/raphink/gists{/gist_id}","starred_url":"https://api.github.com/users/raphink/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/raphink/subscriptions","organizations_url":"https://api.github.com/users/raphink/orgs","repos_url":"https://api.github.com/users/raphink/repos","events_url":"https://api.github.com/users/raphink/events{/privacy}","received_events_url":"https://api.github.com/users/raphink/received_events","type":"User","site_admin":false},"created_at":"2015-08-31T21:46:36Z","updated_at":"2015-08-31T21:46:36Z","body":"It'd be really great if you could provide a failing unit test for this."}}