department-of-veterans-affairs / notification-api

Notification API
MIT License
16 stars 9 forks source link

Notify Incoming SMS Lambda - Altered SSL Context #1012

Open k-macmillan opened 1 year ago

k-macmillan commented 1 year ago

Value Statement

The lambda currently has verify=False setup due to the way requests sets up TLS. We likely need a refined SSL context to allow intranet -> internet.

Acceptance Criteria

GIVEN The notify incoming sms lambda triggers WHEN making POST requests to endpoints THEN those POST requests will be secured

Checklist

Assumptions

Additional Info/Resources

k-macmillan commented 2 weeks ago

Still an issue, keeping.