Merge pull request #3431 from aws/release-v1.81.0

Release 1.81.0 (to main)

Author: Connor Robertson

integration/combination/test_connectors.py

@@ -62,6 +62,7 @@ def tearDown(self):
             ("combination/connector_appsync_api_to_lambda",),
             ("combination/connector_appsync_to_lambda",),
             ("combination/connector_appsync_to_table",),
+            ("combination/connector_appsync_to_eventbus",),
             ("combination/connector_function_to_function",),
             ("combination/connector_restapi_to_function",),
             ("combination/connector_httpapi_to_function",),

integration/resources/expected/combination/connector_appsync_to_eventbus.json

@@ -0,0 +1,38 @@
+[
+  {
+    "LogicalResourceId": "ApiKey",
+    "ResourceType": "AWS::AppSync::ApiKey"
+  },
+  {
+    "LogicalResourceId": "ApiSchema",
+    "ResourceType": "AWS::AppSync::GraphQLSchema"
+  },
+  {
+    "LogicalResourceId": "AppSyncApi",
+    "ResourceType": "AWS::AppSync::GraphQLApi"
+  },
+  {
+    "LogicalResourceId": "AppSyncEventBusDataSource",
+    "ResourceType": "AWS::AppSync::DataSource"
+  },
+  {
+    "LogicalResourceId": "AppSyncSayHelloResolver",
+    "ResourceType": "AWS::AppSync::Resolver"
+  },
+  {
+    "LogicalResourceId": "ConnectorPolicy",
+    "ResourceType": "AWS::IAM::ManagedPolicy"
+  },
+  {
+    "LogicalResourceId": "EventBridgeRole",
+    "ResourceType": "AWS::IAM::Role"
+  },
+  {
+    "LogicalResourceId": "EventBus",
+    "ResourceType": "AWS::Events::EventBus"
+  },
+  {
+    "LogicalResourceId": "TriggerFunction",
+    "ResourceType": "AWS::Lambda::Function"
+  }
+]

integration/resources/templates/combination/connector_appsync_to_eventbus.yaml

@@ -0,0 +1,192 @@
+Resources:
+  EventBus:
+    Type: AWS::Events::EventBus
+    Properties:
+      Name: !Sub "${AWS::StackName}-EventBus"
+
+  EventBridgeRole:
+    Type: AWS::IAM::Role
+    Properties:
+      AssumeRolePolicyDocument:
+        Version: '2012-10-17'
+        Statement:
+        - Effect: Allow
+          Action:
+          - sts:AssumeRole
+          Principal:
+            Service:
+            - appsync.amazonaws.com
+            - lambda.amazonaws.com
+
+  AppSyncApi:
+    Type: AWS::AppSync::GraphQLApi
+    Properties:
+      Name: AppSyncApi
+      AuthenticationType: API_KEY
+
+  ApiSchema:
+    Type: AWS::AppSync::GraphQLSchema
+    Properties:
+      ApiId: !GetAtt AppSyncApi.ApiId
+      Definition: |
+        type EntryDetails {
+          ErrorCode: String
+          ErrorMessage: String
+          EventId: String!
+        }
+
+        type PutEventsResult {
+          Entries: [EntryDetails!]!
+          FailedEntry: Int
+        }
+
+        type Query {
+          sayHello: PutEventsResult!
+        }
+
+        schema {
+          query: Query
+        }
+
+  AppSyncEventBusDataSource:
+    Type: AWS::AppSync::DataSource
+    Properties:
+      ApiId: !GetAtt AppSyncApi.ApiId
+      Name: AppSyncEventBusDataSource
+      Type: AMAZON_EVENTBRIDGE
+      ServiceRoleArn: !GetAtt EventBridgeRole.Arn
+      EventBridgeConfig:
+        EventBusArn: !GetAtt 'EventBus.Arn'
+
+  AppSyncSayHelloResolver:
+    DependsOn: ApiSchema
+    Type: AWS::AppSync::Resolver
+    Properties:
+      ApiId: !GetAtt AppSyncApi.ApiId
+      TypeName: Query
+      FieldName: sayHello
+      DataSourceName: !GetAtt AppSyncEventBusDataSource.Name
+      Runtime:
+        Name: APPSYNC_JS
+        RuntimeVersion: 1.0.0
+      Code: |
+        import { util } from '@aws-appsync/utils';
+        export function request(ctx) {
+          return {
+            "operation" : "PutEvents",
+            "events" : [{
+              "source": "com.mycompany.myapp",
+              "detail": {
+                "key1" : "value1",
+                "key2" : "value2"
+              },
+              "resources": ["Resource1", "Resource2"],
+              "detailType": "myDetailType"
+            }]
+          }
+        }
+
+        export function response(ctx) {
+          if(ctx.error)
+            util.error(ctx.error.message, ctx.error.type, ctx.result)
+          else
+            return ctx.result
+        }
+
+  Connector:
+    Type: AWS::Serverless::Connector
+    Properties:
+      Source:
+        Id: AppSyncEventBusDataSource
+      Destination:
+        Id: EventBus
+      Permissions:
+      - Write
+
+  ApiKey:
+    Type: AWS::AppSync::ApiKey
+    Properties:
+      ApiId: !GetAtt AppSyncApi.ApiId
+
+  TriggerFunction:
+    Type: AWS::Serverless::Function
+    Properties:
+      Role: !GetAtt EventBridgeRole.Arn
+      Environment:
+        Variables:
+          API_KEY: !GetAtt ApiKey.ApiKey
+          GRAPHQL_URL: !GetAtt AppSyncApi.GraphQLUrl
+          EventBusName: !Ref EventBus
+      Runtime: nodejs16.x
+      Handler: index.handler
+      InlineCode: |
+        const https = require("https");
+
+        exports.handler = async () => {
+          const queries = {
+            sayHello: /* GraphQL */ `
+              query {
+                sayHello {
+                  Entries {
+                    ErrorCode
+                    EventId
+                    ErrorMessage
+                  }
+                  FailedEntry
+                }
+              }
+            `,
+          };
+
+          const fetch = async (url, options) =>
+            new Promise((resolve, reject) => {
+              const req = https.request(url, options, (res) => {
+                const body = [];
+                res.on("data", (chunk) => body.push(chunk));
+                res.on("end", () => {
+                  const resString = Buffer.concat(body).toString();
+                  resolve(resString);
+                });
+              });
+              req.on("error", (err) => {
+                reject(err);
+              });
+              req.on("timeout", () => {
+                req.destroy();
+                reject(new Error("Request time out"));
+              });
+              req.write(options.body);
+              req.end();
+            });
+
+          const makeRequest = async (queryName) => {
+            const options = {
+              method: "POST",
+              headers: {
+                "x-api-key": process.env.API_KEY,
+              },
+              body: JSON.stringify({ query: queries[queryName] }),
+              timeout: 600000, // ms
+            };
+
+            const response = await fetch(process.env.GRAPHQL_URL, options);
+            let body = JSON.parse(response);
+            const data = body.data?.[queryName];
+            
+            if (body.errors !== undefined) {
+              throw JSON.stringify(body.errors);
+            }
+            
+            if (data.FailedEntry != null || data.ErrorCode != null ) {
+              throw new Error(
+                `${queryName} error: failed to send event to eventbus ${process.env.EventBusName}`);
+            }
+            
+            return body.data;
+          };
+
+          await makeRequest("sayHello");
+        };
+
+Metadata:
+  SamTransformTest: true

integration/single/test_basic_state_machine.py

@@ -34,6 +34,10 @@ def test_basic_state_machine_with_tags(self):
         self._verify_tag_presence(tags, "TagOne", "ValueOne")
         self._verify_tag_presence(tags, "TagTwo", "ValueTwo")
 
+    @skipIf(
+        current_region_does_not_support([STATE_MACHINE_INLINE_DEFINITION]),
+        "StateMachine with inline definition is not supported in this testing region",
+    )
     def test_state_machine_with_role_path(self):
         """
         Creates a State machine with a Role Path

pytest.ini

@@ -20,3 +20,5 @@ filterwarnings =
     ignore:The --rsyncdir command line argument and rsyncdirs config variable are deprecated.:DeprecationWarning
     # Pytest warnings
     ignore::pytest.PytestUnraisableExceptionWarning
+    # https://github.com/urllib3/urllib3/blob/main/src/urllib3/poolmanager.py#L313
+    ignore::DeprecationWarning:urllib3.*:

samtranslator/__init__.py

@@ -1 +1 @@
-__version__ = "1.80.0"
+__version__ = "1.81.0"

samtranslator/internal/schema_source/aws_serverless_function.py

@@ -628,7 +628,7 @@ class Properties(BaseModel):
     SnapStart: Optional[SnapStart] = prop("SnapStart")
     RuntimeManagementConfig: Optional[RuntimeManagementConfig] = prop("RuntimeManagementConfig")
     Tags: Optional[Tags] = prop("Tags")
-    PropagateTags: Optional[bool]  # TODO: add docs
+    PropagateTags: Optional[bool] = prop("PropagateTags")
     Timeout: Optional[Timeout] = prop("Timeout")
     Tracing: Optional[Tracing] = prop("Tracing")
     VersionDescription: Optional[PassThroughProp] = prop("VersionDescription")
@@ -659,7 +659,7 @@ class Globals(BaseModel):
         ["AWS::Lambda::Function", "Properties", "Environment"],
     )
     Tags: Optional[Tags] = prop("Tags")
-    PropagateTags: Optional[bool]  # TODO: add docs
+    PropagateTags: Optional[bool] = prop("PropagateTags")
     Tracing: Optional[Tracing] = prop("Tracing")
     KmsKeyArn: Optional[KmsKeyArn] = prop("KmsKeyArn")
     Layers: Optional[Layers] = prop("Layers")

samtranslator/internal/schema_source/common.py

@@ -99,3 +99,4 @@ class ResourceAttributes(BaseModel):
     Metadata: Optional[PassThroughProp]
     UpdateReplacePolicy: Optional[PassThroughProp]
     Condition: Optional[PassThroughProp]
+    IgnoreGlobals: Optional[Union[str, List[str]]]

samtranslator/model/connector_profiles/profiles.json

@@ -790,6 +790,23 @@
             }
           }
         }
+      },
+      "AWS::Events::EventBus": {
+        "Type": "AWS_IAM_ROLE_MANAGED_POLICY",
+        "Properties": {
+          "SourcePolicy": true,
+          "AccessCategories": {
+            "Write": {
+              "Statement": [
+                {
+                  "Effect": "Allow",
+                  "Action": ["events:PutEvents"],
+                  "Resource": ["%{Destination.Arn}"]
+                }
+              ]
+            }
+          }
+        }
       }
     },
     "AWS::AppSync::GraphQLApi": {

samtranslator/model/eventsources/push.py

@@ -1116,8 +1116,11 @@ def add_auth_to_swagger(  # noqa: PLR0912, PLR0913
                         )
 
         auth_scopes = event_auth.get("AuthorizationScopes")
+
         if auth_scopes:
             sam_expect(auth_scopes, event_id, "Auth.AuthorizationScopes", is_sam_event=True).to_be_a_list()
+            if not method_authorizer:
+                raise InvalidEventException(event_id, "AuthorizationScopes works only when Authorizer is set")
 
         apikey_required_setting = event_auth.get("ApiKeyRequired")
         apikey_required_setting_is_false = apikey_required_setting is not None and not apikey_required_setting