Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Bucket AccessControl to allow Ref #475

Merged
merged 1 commit into from
Apr 30, 2016

Conversation

jonapich
Copy link
Contributor

The current implementation will break when attempting to "Ref" the AccessControl to a stack parameter.

@markpeek markpeek merged commit 6afee75 into cloudtools:master Apr 30, 2016
@markpeek
Copy link
Member

Thanks!

webratz pushed a commit to webratz/troposphere that referenced this pull request May 4, 2016
* cloudtools/master: (80 commits)
  Add test for AWS::Logs::Destination (cloudtools#482)
  Fix baseclass for AWS::Logs::Destination (Fixes cloudtools#481)
  Update for changes since 1.5.0
  Fix Bucket AccessControl to allow Ref (cloudtools#475)
  Better AWSHelperFn support in template generator (cloudtools#473)
  Implement LifecycleRule Transitions property (cloudtools#472)
  Update to Apr 25, 2016 release (cloudtools#471)
  Fix Source object to take a list of SourceDetails (cloudtools#469)
  Update Contribute Document to Use Requirements.txt (cloudtools#470)
  Add WAF SizeConstraintSet and XssMatchSet
  Add backward compatibility for the deprecated ElasticsearchDomain
  Fix Elasticsarch Domain object naming
  Import JSON Templates (cloudtools#468)
  ReplicationConfigurationRules Destination is now an object (cloudtools#380)
  Remove unnecessary AWSHelperFn from props
  EC2 update FromPort, ToPort and Egress as optional
  Added CloudWatch Events support
  ApiGateway Resources (cloudtools#466)
  Provide Valid Lambda Function Memory Values for use in Parameters (cloudtools#449)
  move Groups to property, add policy template version (cloudtools#460)
  ...
amosshapira pushed a commit to amosshapira/troposphere that referenced this pull request Oct 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants