lucid-cumulus 0.11.5 → 0.11.6

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NjcwMDAwZjA2YzQ5OTAxY2UwNmU2MzAwNTk2ZTIxZmU5NDNlM2RhOQ==
+    Y2FhNDkzYjc3ZDQzOTgwYjhhYzMxZTI0Y2U5NGM0MzRmYjYyM2JjNg==
   data.tar.gz: !binary |-
-    YjA1NmFiYjg4YzlkNmVhNzg1MjBmN2I3N2M4OTFkOGY4ZTkwMjZmZQ==
+    NGQzNjUzZGIyYjhkZWI1ZGNjZDJkNjk5OTg4NTI2MjhjMGIxMmE3MA==
 SHA512:
   metadata.gz: !binary |-
-    NGY4OTQ3NTk0NWZkOTM1MmFmZTc2YWM1Mzg3YzY4NmM3NGI2ZmFhY2IyODhk
-    YzMzOTE5NzhhZmQ5OGQ5OWNlYmEyYTBkNDEwMDIyNjI2NTJhNDA0N2U5YmMz
-    OTZjNzIwNzQ2MjdkMzFmODYyMjFiMTBkMjMyNzU0MDdhZWM3Y2U=
+    OGQzNGFiODQ5ZWJhYTg1YWM2NGE2YTUxZDFkMGExYTVhZTYzNjJkYzQ2OTFh
+    Zjg1YmYzMWZkZWE3NDNiNGQ0NjRkZjc4OTU3OTUxMzNmNWQyMjU5ZGRkZDFl
+    NGY1NTBkZjU5YjQxZDIzZTg4MDg0MmFlNDBjZTIzMWQ4ZTc4ZjQ=
   data.tar.gz: !binary |-
-    MmM2ZmI2NjRjZTI4OTU5NGU3MDM0OGJhZTljOWYwMTQ4ZWJmYTA3ZDdmODAy
-    YWVhNGQ3NjQzMTZhMmQxODM2ZDlmNGM5NmE2ZjU3ODRkODg0NjU5NWJiNjc0
-    YWUwNWUwZGNlZjkxNDc1MGI5YWYxOTRlZTNlZjE3ZjhiZDM4MTY=
+    MjgyZjhmZTBmODRjMzc5NjZmZDM2MjA1ZTkyYmIxNzY5MmNmMzM5OTdkMzc3
+    ZWZjMWYwM2M2MWRhN2E1NzEwNzRiODMzODVjZWNlNDMwOWE3ZjU2ODRmYzg5
+    NjU4MThhMThlNTE3MmJmMjg3NGNjMDdhMjMzNjcyOTRmOTU4MjA=

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    lucid-cumulus (0.11.5)
+    lucid-cumulus (0.11.6)
       aws-sdk (= 2.2.30)
       deepsort (~> 0.1)
       parse-cron (~> 0.1.4)
@@ -15,7 +15,7 @@ GEM
       jmespath (~> 1.0)
     aws-sdk-resources (2.2.30)
       aws-sdk-core (= 2.2.30)
-    deepsort (0.2.1)
+    deepsort (0.3.1)
     diff-lcs (1.2.5)
     jmespath (1.3.1)
     parse-cron (0.1.4)

data/lib/aws_extensions/iam/Policy.rb

@@ -6,7 +6,7 @@ module AwsExtensions
     module Policy
       def as_hash
         # Sort the statments to prevent false conflicts while diffing
-        sorted_policy = JSON.parse(URI.unescape(policy_document)).deep_sort
+        sorted_policy = JSON.parse(URI.unescape(policy_document))
         sorted_policy["Statement"].each do |statement|
           # actions sometimes contains a single string element instead of the expected array
           statement["Action"] = [statement["Action"]] if statement["Action"].is_a? String
@@ -14,7 +14,7 @@ module AwsExtensions
           statement["Resource"] = [statement["Resource"]] if statement["Resource"].is_a? String
         end
         # return the sorted policy hash
-        sorted_policy
+        sorted_policy.deep_sort
       end
     end
   end

data/lib/iam/models/ResourceWithPolicy.rb

@@ -192,8 +192,7 @@ module Cumulus
           else
             aws_statements = aws_policy["Statement"]
             local_statements = p.as_hash["Statement"]
-            # turns out, policies defined locally can be redundant. this causes false diffs
-            if aws_statements.uniq != local_statements.uniq
+            if aws_statements != local_statements
               diff = IamDiff.new(IamChange::POLICY, aws_statements, p)
               diff.policy_name = generated_policy_name
               diffs << diff

data/lucid-cumulus.gemspec

@@ -4,7 +4,7 @@ require "bundler"
 
 Gem::Specification.new do |s|
   s.name                      = "lucid-cumulus"
-  s.version                   = "0.11.5"
+  s.version                   = "0.11.6"
   s.platform                  = Gem::Platform::RUBY
   s.authors                   = ["Keilan Jackson", "Mark Siebert"]
   s.email                     = "cumulus@lucidchart.com"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lucid-cumulus
 version: !ruby/object:Gem::Version
-  version: 0.11.5
+  version: 0.11.6
 platform: ruby
 authors:
 - Keilan Jackson
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-08-16 00:00:00.000000000 Z
+date: 2016-08-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-sdk