Prevent "testing" user from being on packet #365

Activity

1. mxmeinhold commented on Sep 25, 2023

   

   mxmeinhold

   on Sep 25, 2023

   Collaborator

   More actions

   We pull all the upperclassmen at a couple of points. One is when creating new packets, another is in sync_with_ldap. Both of these rely on get_active_members

   There are a few cases where the testing user can be useful to have around, such as in dev or staging, but these aren't super important. The simplest solution I think would be to exclude the testing user from the ldap responses entirely. A slightly more complex solution that would still allow for testing would be to exclude the testing user only when deployed in production. This could be something handled by an environment/config variable, or it could be something we change in the actual image build. Lots of options!