department-of-veterans-affairs / notification-api

Notification API
MIT License
16 stars 9 forks source link

Upgrade the Python requests package in requirements-lambda.txt files #1329

Closed kalbfled closed 1 month ago

kalbfled commented 1 year ago

User Story - Business Need

User Story(ies)

As a Notify developer I want lambda functions to use the latest version of the Python "requests" package So that we minimize security vulnerabilities.

Additional Info and Resources

The lambda functions that use the "requests" Python package are:

...notification-api/lambda_functions$ grep requests */requirements-lambda.txt
two_way_sms/requirements-lambda.txt:requests ~= 2.28.2
user_flows/requirements-lambda.txt:requests~=2.29.0
vetext_incoming_forwarder_lambda/requirements-lambda.txt:requests ~= 2.28.2

Altering these requirements files is for documentation only. The lambda functions actually use the "requests" lambda layer. See vanotify-infra 585.

Engineering Checklist

Acceptance Criteria

QA Considerations

N/A

Potential Dependencies

Leave blank if n/a

Out of Scope

Leave blank if n/a

mjones-oddball commented 1 year ago

Hey team! Please add your planning poker estimate with Zenhub @cris-oddball @EvanParish @justaskdavidb2 @k-macmillan @kalbfled @ldraney @nikolai-efimov

npmartin-oddball commented 1 month ago

Not needed.