Skip to content

Conversation

@bachtran02
Copy link

@bachtran02 bachtran02 commented Mar 28, 2025

Changes Summary:

Changes in email.py

  • replace hardcoded STMP server credentials with environment variables configured in Cloud Run.
  • emails are now sent correctly so cleaned up commented-out code.

Added course name and assignment title to assignment extension logging.

Course name can be configured with key COURSE_NAME in Spreadsheet Environment Variables sheet.
It is defaulted to empty string if not configured.

Documentation in Extension Request

Documentation info is added to Slack message if provided by student.

service-account.json

In Cloud Source repo, service-account.json file is included directly in the repo so that the functions can access the spreadsheets. This credential file has been moved to Google Secret Manager and mounted to the Cloud Run functions at /var/secrets/service-account-json at runtime. Existing staging functions have been configured to handle this.

All changes to hide credentials/private data is meant to be carried out on developer's side. People who create copy of spreadsheet for courses (e.g TAs) should not worry about any of these.

Testing

Updates to testing functions to dynamically test if policy.apply() works correctly.

@cycomachead cycomachead merged commit 43e68e1 into master Jun 14, 2025
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Updating EMAIL_FROM variable in Environment Variables (GS) Ensure all sensitive data is in secrets/OS environment variables

3 participants