Bug: publication_approved never accesses errors param
Nobody has claimed this yet.
Assessment
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 25/100
Research direction
Start by locating the lambda_handler entry point and the publication_approved method, then trace how the errors value is passed between them. The issue does not state what publication_approved should do with errors, so confirm the intended behavior before changing code; done means the parameter has a defined, tested use or is removed consistently.
Written by the indexing model from the issue text.
Description
Expected Behaviour
publication_approved method never access errors
Current Behaviour
No issue, parameter is never used
Code snippet
def lambda_handler(event, context):
"""Main entry point for Property Approval lambda function
Parameters
----------
event : API Gateway Lambda Proxy Request
The event passed to the function.
context : AWS Lambda Context
The context for the Lambda function.
Returns
-------
Success message upon successful storage of the approval outcome into DynamoDB
"""
logger.info(event)
errors = None if 'workflowErrors' not in event['detail'] else event['detail'].pop('workflowErrors')
# Deserialize event into strongly typed object
awsEvent:AWSEvent = Marshaller.unmarshall(event, AWSEvent) # type: ignore
detail:PublicationEvaluationCompleted = awsEvent.detail # type: ignore
return publication_approved(detail, errors)def lambda_handler(event, context):
"""Main entry point for Property Approval lambda function
Parameters
----------
event : API Gateway Lambda Proxy Request
The event passed to the function.
context : AWS Lambda Context
The context for the Lambda function.
Returns
-------
Success message upon successful storage of the approval outcome into DynamoDB
"""
logger.info(event)
errors = None if 'workflowErrors' not in event['detail'] else event['detail'].pop('workflowErrors')
# Deserialize event into strongly typed object
awsEvent:AWSEvent = Marshaller.unmarshall(event, AWSEvent) # type: ignore
detail:PublicationEvaluationCompleted = awsEvent.detail # type: ignore
return publication_approved(detail, errors)
Possible Solution
No response
Steps to Reproduce
N/A
Debugging logs
No response
- Dominant language
- Python
- Stars
- 47
- Forks
- 20
- PR merge metrics
- No merged PRs in 30d
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from aws-samples/aws-serverless-developer-experience-workshop-python
-
pending-release
Difficulty 4/5 3-5 days Newbie friendliness 35/100
aws-samples/aws-serverless-developer-experience-workshop-python#178 ·
-
internal pending-release triage
aws-samples/aws-serverless-developer-experience-workshop-python#32 · 1 assignee ·
All issues in aws-samples/aws-serverless-developer-experience-workshop-python
Similar issues
-
essnmx good first issue
Difficulty 1/5 Under an hour Newbie friendliness 95/100
-
[Feature] 奇物选择添加优先级 Open
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
syfoud/Simulated_Scepter#174 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Giskard-AI/giskard-oss#2840 · 1 comment ·
-
A claim comment carrying the issue number is silently declined while the workflow reports success Openarea: repo bug perceived difficulty: 2
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
yeti-platform/yeti#1380 ·