cds-snc / notification-planning

Project planning for GC Notify Team
5 stars 0 forks source link

Get a 500 error going to /services/.../notifications/all #1662

Open sastels opened 2 months ago

sastels commented 2 months ago

Describe the bug

Admin throws a 500 error going to, for example,

https://staging.notification.cdssandbox.xyz/services/d6aa2c68-a2d9-4437-ab19-3ae8eb202553/notifications/all

Bug Severity

See examples in the documentation

SEV-4 Low - can only do this for a valid service that you have access to.

To Reproduce

Steps to reproduce the behavior:

  1. Log into a platform admin account on staging
  2. Go to https://staging.notification.cdssandbox.xyz/services/d6aa2c68-a2d9-4437-ab19-3ae8eb202553/notifications/steve
  3. See error.

Expected behavior

not an 500. Maybe a 404.

Impact

Too many 500s will trigger an ops genie alert.

Other information

Admin does an api call, and api returns an error {'template_type': ['Must be one of: sms, email, letter.']}}}

sastels commented 2 months ago

These might be coming from Notify redirects somehow... I see in the logs)

96.21.68.132 - - [05/Sep/2024:13:07:06 +0000] "GET /sign-in?next=/services/d6aa2c68-a2d9-4437-ab19-3ae8eb202553/notifications/all HTTP/1.1" 200 82362