We are currently solving the following issue within our repository. Here is the issue text:
--- BEGIN ISSUE ---
Update rule E2541 to not error when action names are not strings
*Issue #, if available:*
Fix #752
*Description of changes:*
- Update rule E2541 to only check for duplicate names when the action name is a string

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.


--- END ISSUE ---

Below are some code segments, each from a relevant file. One or more of these files may contain bugs.
--- BEGIN FILES ---
Path: `src/cfnlint/transform.py`
Content:
```
1 """
2   Copyright 2019 Amazon.com, Inc. or its affiliates. All Rights Reserved.
3 
4   Permission is hereby granted, free of charge, to any person obtaining a copy of this
5   software and associated documentation files (the "Software"), to deal in the Software
6   without restriction, including without limitation the rights to use, copy, modify,
7   merge, publish, distribute, sublicense, and/or sell copies of the Software, and to
8   permit persons to whom the Software is furnished to do so.
9 
10   THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,
11   INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A
12   PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT
13   HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
14   OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
15   SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
16 """
17 import os
18 import logging
19 import six
20 import samtranslator
21 from samtranslator.parser import parser
22 from samtranslator.translator.translator import Translator
23 from samtranslator.public.exceptions import InvalidDocumentException
24 
25 import cfnlint.helpers
26 LOGGER = logging.getLogger('cfnlint')
27 
28 class Transform(object):
29     """
30     Application Serverless Module tranform Wrappor. Based on code from AWS SAM CLI:
31     https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/commands/validate/lib/sam_template_validator.py
32     """
33 
34     def __init__(self, filename, template, region):
35         """
36         Initialize Transform class
37         """
38         self._filename = filename
39         self._template = template
40         self._region = region
41 
42         self._managed_policy_map = self.load_managed_policies()
43         self._sam_parser = parser.Parser()
44 
45     def template(self):
46         """Get the template"""
47         return self._template
48 
49     def load_managed_policies(self):
50         """
51         Load the ManagedPolicies locally, based on the AWS-CLI:
52         https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/lib/samlib/default_managed_policies.json
53         """
54         return cfnlint.helpers.load_resources('data/Serverless/ManagedPolicies.json')
55 
56     def _replace_local_codeuri(self):
57         """
58         Replaces the CodeUri in AWS::Serverless::Function and DefinitionUri in AWS::Serverless::Api to a fake
59         S3 Uri. This is to support running the SAM Translator with valid values for these fields. If this in not done,
60         the template is invalid in the eyes of SAM Translator (the translator does not support local paths)
61         """
62 
63         all_resources = self._template.get('Resources', {})
64 
65         for _, resource in all_resources.items():
66 
67             resource_type = resource.get('Type')
68             resource_dict = resource.get('Properties')
69 
70             if resource_type == 'AWS::Serverless::Function':
71 
72                 Transform._update_to_s3_uri('CodeUri', resource_dict)
73             if resource_type in ['AWS::Serverless::LayerVersion']:
74                 if resource_dict.get('ContentUri'):
75                     Transform._update_to_s3_uri('ContentUri', resource_dict)
76             if resource_type == 'AWS::Serverless::Application':
77                 if resource_dict.get('Location'):
78                     resource_dict['Location'] = ''
79                     Transform._update_to_s3_uri('Location', resource_dict)
80             if resource_type == 'AWS::Serverless::Api':
81                 if 'DefinitionBody' not in resource_dict:
82                     Transform._update_to_s3_uri('DefinitionUri', resource_dict)
83 
84     def transform_template(self):
85         """
86         Transform the Template using the Serverless Application Model.
87         """
88         matches = []
89 
90         try:
91             # Output the SAM Translator version in debug mode
92             LOGGER.debug('SAM Translator: %s', samtranslator.__version__)
93 
94             sam_translator = Translator(managed_policy_map=self._managed_policy_map,
95                                         sam_parser=self._sam_parser)
96 
97             self._replace_local_codeuri()
98 
99             # Tell SAM to use the region we're linting in, this has to be controlled using the default AWS mechanisms, see also:
100             # https://github.com/awslabs/serverless-application-model/blob/master/samtranslator/translator/arn_generator.py
101             LOGGER.debug('Setting AWS_DEFAULT_REGION to %s', self._region)
102             os.environ['AWS_DEFAULT_REGION'] = self._region
103 
104             self._template = cfnlint.helpers.convert_dict(
105                 sam_translator.translate(sam_template=self._template, parameter_values={}))
106 
107             LOGGER.debug('Transformed template: %s', self._template)
108         except InvalidDocumentException as e:
109             message = 'Error transforming template: {0}'
110             for cause in e.causes:
111                 matches.append(cfnlint.Match(
112                     1, 1,
113                     1, 1,
114                     self._filename, cfnlint.TransformError(), message.format(cause.message)))
115         except Exception as e:  # pylint: disable=W0703
116             LOGGER.debug('Error transforming template: %s', str(e))
117             LOGGER.debug('Stack trace: %s', e, exc_info=True)
118             message = 'Error transforming template: {0}'
119             matches.append(cfnlint.Match(
120                 1, 1,
121                 1, 1,
122                 self._filename, cfnlint.TransformError(), message.format(str(e))))
123 
124         return matches
125 
126     @staticmethod
127     def is_s3_uri(uri):
128         """
129         Checks the uri and determines if it is a valid S3 Uri
130         Parameters
131         ----------
132         uri str, required
133             Uri to check
134         Returns
135         -------
136         bool
137             Returns True if the uri given is an S3 uri, otherwise False
138         """
139         return isinstance(uri, six.string_types) and uri.startswith('s3://')
140 
141     @staticmethod
142     def _update_to_s3_uri(property_key, resource_property_dict, s3_uri_value='s3://bucket/value'):
143         """
144         Updates the 'property_key' in the 'resource_property_dict' to the value of 's3_uri_value'
145         Note: The function will mutate the resource_property_dict that is pass in
146         Parameters
147         ----------
148         property_key str, required
149             Key in the resource_property_dict
150         resource_property_dict dict, required
151             Property dictionary of a Resource in the template to replace
152         s3_uri_value str, optional
153             Value to update the value of the property_key to
154         """
155         uri_property = resource_property_dict.get(property_key, '.')
156 
157         # ignore if dict or already an S3 Uri
158         if isinstance(uri_property, dict) or Transform.is_s3_uri(uri_property):
159             return
160 
161         resource_property_dict[property_key] = s3_uri_value
162
```
--- END FILES ---

Please first localize the bug based on the issue statement, and then generate a patch according to the `git diff` format fenced by three backticks.

Here is an example:

```diff
diff --git a/examples/server_async.py b/examples/server_async.py
--- a/examples/server_async.py
+++ b/examples/server_async.py
@@ -313,4 +313,4 @@
 
 
 if __name__ == "__main__":
-    asyncio.run(run_async_server("."), debug=True)
+    asyncio.run(run_async_server(), debug=True)
diff --git a/examples/server_sync.py b/examples/server_sync.py
--- a/examples/server_sync.py
+++ b/examples/server_sync.py
@@ -313,5 +313,5 @@
 
 
 if __name__ == "__main__":
-    server = run_sync_server(".")
+    server = run_sync_server()
     server.shutdown()

```

diff --git a/src/cfnlint/transform.py b/src/cfnlint/transform.py
--- a/src/cfnlint/transform.py
+++ b/src/cfnlint/transform.py
@@ -78,8 +78,10 @@
                     resource_dict['Location'] = ''
                     Transform._update_to_s3_uri('Location', resource_dict)
             if resource_type == 'AWS::Serverless::Api':
-                if 'DefinitionBody' not in resource_dict:
+                if 'DefinitionBody' not in resource_dict and 'Auth' not in resource_dict:
                     Transform._update_to_s3_uri('DefinitionUri', resource_dict)
+                else:
+                    resource_dict['DefinitionBody'] = ''
 
     def transform_template(self):
         """

{"golden_diff": "diff --git a/src/cfnlint/transform.py b/src/cfnlint/transform.py\n--- a/src/cfnlint/transform.py\n+++ b/src/cfnlint/transform.py\n@@ -78,8 +78,10 @@\n                     resource_dict['Location'] = ''\n                     Transform._update_to_s3_uri('Location', resource_dict)\n             if resource_type == 'AWS::Serverless::Api':\n-                if 'DefinitionBody' not in resource_dict:\n+                if 'DefinitionBody' not in resource_dict and 'Auth' not in resource_dict:\n                     Transform._update_to_s3_uri('DefinitionUri', resource_dict)\n+                else:\n+                    resource_dict['DefinitionBody'] = ''\n \n     def transform_template(self):\n         \"\"\"\n", "issue": "Update rule E2541 to not error when action names are not strings\n*Issue #, if available:*\r\nFix #752\r\n*Description of changes:*\r\n- Update rule E2541 to only check for duplicate names when the action name is a string\r\n\r\nBy submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.\r\n\n", "before_files": [{"content": "\"\"\"\n  Copyright 2019 Amazon.com, Inc. or its affiliates. All Rights Reserved.\n\n  Permission is hereby granted, free of charge, to any person obtaining a copy of this\n  software and associated documentation files (the \"Software\"), to deal in the Software\n  without restriction, including without limitation the rights to use, copy, modify,\n  merge, publish, distribute, sublicense, and/or sell copies of the Software, and to\n  permit persons to whom the Software is furnished to do so.\n\n  THE SOFTWARE IS PROVIDED \"AS IS\", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,\n  INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A\n  PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT\n  HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION\n  OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE\n  SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.\n\"\"\"\nimport os\nimport logging\nimport six\nimport samtranslator\nfrom samtranslator.parser import parser\nfrom samtranslator.translator.translator import Translator\nfrom samtranslator.public.exceptions import InvalidDocumentException\n\nimport cfnlint.helpers\nLOGGER = logging.getLogger('cfnlint')\n\nclass Transform(object):\n    \"\"\"\n    Application Serverless Module tranform Wrappor. Based on code from AWS SAM CLI:\n    https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/commands/validate/lib/sam_template_validator.py\n    \"\"\"\n\n    def __init__(self, filename, template, region):\n        \"\"\"\n        Initialize Transform class\n        \"\"\"\n        self._filename = filename\n        self._template = template\n        self._region = region\n\n        self._managed_policy_map = self.load_managed_policies()\n        self._sam_parser = parser.Parser()\n\n    def template(self):\n        \"\"\"Get the template\"\"\"\n        return self._template\n\n    def load_managed_policies(self):\n        \"\"\"\n        Load the ManagedPolicies locally, based on the AWS-CLI:\n        https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/lib/samlib/default_managed_policies.json\n        \"\"\"\n        return cfnlint.helpers.load_resources('data/Serverless/ManagedPolicies.json')\n\n    def _replace_local_codeuri(self):\n        \"\"\"\n        Replaces the CodeUri in AWS::Serverless::Function and DefinitionUri in AWS::Serverless::Api to a fake\n        S3 Uri. This is to support running the SAM Translator with valid values for these fields. If this in not done,\n        the template is invalid in the eyes of SAM Translator (the translator does not support local paths)\n        \"\"\"\n\n        all_resources = self._template.get('Resources', {})\n\n        for _, resource in all_resources.items():\n\n            resource_type = resource.get('Type')\n            resource_dict = resource.get('Properties')\n\n            if resource_type == 'AWS::Serverless::Function':\n\n                Transform._update_to_s3_uri('CodeUri', resource_dict)\n            if resource_type in ['AWS::Serverless::LayerVersion']:\n                if resource_dict.get('ContentUri'):\n                    Transform._update_to_s3_uri('ContentUri', resource_dict)\n            if resource_type == 'AWS::Serverless::Application':\n                if resource_dict.get('Location'):\n                    resource_dict['Location'] = ''\n                    Transform._update_to_s3_uri('Location', resource_dict)\n            if resource_type == 'AWS::Serverless::Api':\n                if 'DefinitionBody' not in resource_dict:\n                    Transform._update_to_s3_uri('DefinitionUri', resource_dict)\n\n    def transform_template(self):\n        \"\"\"\n        Transform the Template using the Serverless Application Model.\n        \"\"\"\n        matches = []\n\n        try:\n            # Output the SAM Translator version in debug mode\n            LOGGER.debug('SAM Translator: %s', samtranslator.__version__)\n\n            sam_translator = Translator(managed_policy_map=self._managed_policy_map,\n                                        sam_parser=self._sam_parser)\n\n            self._replace_local_codeuri()\n\n            # Tell SAM to use the region we're linting in, this has to be controlled using the default AWS mechanisms, see also:\n            # https://github.com/awslabs/serverless-application-model/blob/master/samtranslator/translator/arn_generator.py\n            LOGGER.debug('Setting AWS_DEFAULT_REGION to %s', self._region)\n            os.environ['AWS_DEFAULT_REGION'] = self._region\n\n            self._template = cfnlint.helpers.convert_dict(\n                sam_translator.translate(sam_template=self._template, parameter_values={}))\n\n            LOGGER.debug('Transformed template: %s', self._template)\n        except InvalidDocumentException as e:\n            message = 'Error transforming template: {0}'\n            for cause in e.causes:\n                matches.append(cfnlint.Match(\n                    1, 1,\n                    1, 1,\n                    self._filename, cfnlint.TransformError(), message.format(cause.message)))\n        except Exception as e:  # pylint: disable=W0703\n            LOGGER.debug('Error transforming template: %s', str(e))\n            LOGGER.debug('Stack trace: %s', e, exc_info=True)\n            message = 'Error transforming template: {0}'\n            matches.append(cfnlint.Match(\n                1, 1,\n                1, 1,\n                self._filename, cfnlint.TransformError(), message.format(str(e))))\n\n        return matches\n\n    @staticmethod\n    def is_s3_uri(uri):\n        \"\"\"\n        Checks the uri and determines if it is a valid S3 Uri\n        Parameters\n        ----------\n        uri str, required\n            Uri to check\n        Returns\n        -------\n        bool\n            Returns True if the uri given is an S3 uri, otherwise False\n        \"\"\"\n        return isinstance(uri, six.string_types) and uri.startswith('s3://')\n\n    @staticmethod\n    def _update_to_s3_uri(property_key, resource_property_dict, s3_uri_value='s3://bucket/value'):\n        \"\"\"\n        Updates the 'property_key' in the 'resource_property_dict' to the value of 's3_uri_value'\n        Note: The function will mutate the resource_property_dict that is pass in\n        Parameters\n        ----------\n        property_key str, required\n            Key in the resource_property_dict\n        resource_property_dict dict, required\n            Property dictionary of a Resource in the template to replace\n        s3_uri_value str, optional\n            Value to update the value of the property_key to\n        \"\"\"\n        uri_property = resource_property_dict.get(property_key, '.')\n\n        # ignore if dict or already an S3 Uri\n        if isinstance(uri_property, dict) or Transform.is_s3_uri(uri_property):\n            return\n\n        resource_property_dict[property_key] = s3_uri_value\n", "path": "src/cfnlint/transform.py"}], "after_files": [{"content": "\"\"\"\n  Copyright 2019 Amazon.com, Inc. or its affiliates. All Rights Reserved.\n\n  Permission is hereby granted, free of charge, to any person obtaining a copy of this\n  software and associated documentation files (the \"Software\"), to deal in the Software\n  without restriction, including without limitation the rights to use, copy, modify,\n  merge, publish, distribute, sublicense, and/or sell copies of the Software, and to\n  permit persons to whom the Software is furnished to do so.\n\n  THE SOFTWARE IS PROVIDED \"AS IS\", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED,\n  INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A\n  PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT\n  HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION\n  OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE\n  SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.\n\"\"\"\nimport os\nimport logging\nimport six\nimport samtranslator\nfrom samtranslator.parser import parser\nfrom samtranslator.translator.translator import Translator\nfrom samtranslator.public.exceptions import InvalidDocumentException\n\nimport cfnlint.helpers\nLOGGER = logging.getLogger('cfnlint')\n\nclass Transform(object):\n    \"\"\"\n    Application Serverless Module tranform Wrappor. Based on code from AWS SAM CLI:\n    https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/commands/validate/lib/sam_template_validator.py\n    \"\"\"\n\n    def __init__(self, filename, template, region):\n        \"\"\"\n        Initialize Transform class\n        \"\"\"\n        self._filename = filename\n        self._template = template\n        self._region = region\n\n        self._managed_policy_map = self.load_managed_policies()\n        self._sam_parser = parser.Parser()\n\n    def template(self):\n        \"\"\"Get the template\"\"\"\n        return self._template\n\n    def load_managed_policies(self):\n        \"\"\"\n        Load the ManagedPolicies locally, based on the AWS-CLI:\n        https://github.com/awslabs/aws-sam-cli/blob/develop/samcli/lib/samlib/default_managed_policies.json\n        \"\"\"\n        return cfnlint.helpers.load_resources('data/Serverless/ManagedPolicies.json')\n\n    def _replace_local_codeuri(self):\n        \"\"\"\n        Replaces the CodeUri in AWS::Serverless::Function and DefinitionUri in AWS::Serverless::Api to a fake\n        S3 Uri. This is to support running the SAM Translator with valid values for these fields. If this in not done,\n        the template is invalid in the eyes of SAM Translator (the translator does not support local paths)\n        \"\"\"\n\n        all_resources = self._template.get('Resources', {})\n\n        for _, resource in all_resources.items():\n\n            resource_type = resource.get('Type')\n            resource_dict = resource.get('Properties')\n\n            if resource_type == 'AWS::Serverless::Function':\n\n                Transform._update_to_s3_uri('CodeUri', resource_dict)\n            if resource_type in ['AWS::Serverless::LayerVersion']:\n                if resource_dict.get('ContentUri'):\n                    Transform._update_to_s3_uri('ContentUri', resource_dict)\n            if resource_type == 'AWS::Serverless::Application':\n                if resource_dict.get('Location'):\n                    resource_dict['Location'] = ''\n                    Transform._update_to_s3_uri('Location', resource_dict)\n            if resource_type == 'AWS::Serverless::Api':\n                if 'DefinitionBody' not in resource_dict and 'Auth' not in resource_dict:\n                    Transform._update_to_s3_uri('DefinitionUri', resource_dict)\n                else:\n                    resource_dict['DefinitionBody'] = ''\n\n    def transform_template(self):\n        \"\"\"\n        Transform the Template using the Serverless Application Model.\n        \"\"\"\n        matches = []\n\n        try:\n            # Output the SAM Translator version in debug mode\n            LOGGER.debug('SAM Translator: %s', samtranslator.__version__)\n\n            sam_translator = Translator(managed_policy_map=self._managed_policy_map,\n                                        sam_parser=self._sam_parser)\n\n            self._replace_local_codeuri()\n\n            # Tell SAM to use the region we're linting in, this has to be controlled using the default AWS mechanisms, see also:\n            # https://github.com/awslabs/serverless-application-model/blob/master/samtranslator/translator/arn_generator.py\n            LOGGER.debug('Setting AWS_DEFAULT_REGION to %s', self._region)\n            os.environ['AWS_DEFAULT_REGION'] = self._region\n\n            self._template = cfnlint.helpers.convert_dict(\n                sam_translator.translate(sam_template=self._template, parameter_values={}))\n\n            LOGGER.debug('Transformed template: %s', self._template)\n        except InvalidDocumentException as e:\n            message = 'Error transforming template: {0}'\n            for cause in e.causes:\n                matches.append(cfnlint.Match(\n                    1, 1,\n                    1, 1,\n                    self._filename, cfnlint.TransformError(), message.format(cause.message)))\n        except Exception as e:  # pylint: disable=W0703\n            LOGGER.debug('Error transforming template: %s', str(e))\n            LOGGER.debug('Stack trace: %s', e, exc_info=True)\n            message = 'Error transforming template: {0}'\n            matches.append(cfnlint.Match(\n                1, 1,\n                1, 1,\n                self._filename, cfnlint.TransformError(), message.format(str(e))))\n\n        return matches\n\n    @staticmethod\n    def is_s3_uri(uri):\n        \"\"\"\n        Checks the uri and determines if it is a valid S3 Uri\n        Parameters\n        ----------\n        uri str, required\n            Uri to check\n        Returns\n        -------\n        bool\n            Returns True if the uri given is an S3 uri, otherwise False\n        \"\"\"\n        return isinstance(uri, six.string_types) and uri.startswith('s3://')\n\n    @staticmethod\n    def _update_to_s3_uri(property_key, resource_property_dict, s3_uri_value='s3://bucket/value'):\n        \"\"\"\n        Updates the 'property_key' in the 'resource_property_dict' to the value of 's3_uri_value'\n        Note: The function will mutate the resource_property_dict that is pass in\n        Parameters\n        ----------\n        property_key str, required\n            Key in the resource_property_dict\n        resource_property_dict dict, required\n            Property dictionary of a Resource in the template to replace\n        s3_uri_value str, optional\n            Value to update the value of the property_key to\n        \"\"\"\n        uri_property = resource_property_dict.get(property_key, '.')\n\n        # ignore if dict or already an S3 Uri\n        if isinstance(uri_property, dict) or Transform.is_s3_uri(uri_property):\n            return\n\n        resource_property_dict[property_key] = s3_uri_value\n", "path": "src/cfnlint/transform.py"}]}