Skip to content

241 add auth projects - #285

Merged
Rayna-Yu merged 6 commits into
mainfrom
241-add-auth-projects
Jul 24, 2026
Merged

241 add auth projects#285
Rayna-Yu merged 6 commits into
mainfrom
241-add-auth-projects

Conversation

@Rayna-Yu

Copy link
Copy Markdown
Contributor

#241

Closes #241

📝 Description

  • added authenticateRequest call to the top of the handler
  • added canX auth check to all the appropriate routes
  • added and fixed tests with no auth

✔️ Verification

Added and passed all unit and integration tests.

🏕️ (Optional) Future Work / Notes

N/A

@Rayna-Yu
Rayna-Yu marked this pull request as ready for review July 14, 2026 03:47
@Rayna-Yu
Rayna-Yu requested a review from nourshoreibah as a code owner July 14, 2026 03:47
@github-actions
github-actions Bot requested a review from mehanana July 14, 2026 03:47
github-actions Bot added a commit that referenced this pull request Jul 14, 2026

@mehanana mehanana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!!

@Rayna-Yu
Rayna-Yu force-pushed the 241-add-auth-projects branch from 96416b0 to 05713dd Compare July 23, 2026 23:42
@Rayna-Yu
Rayna-Yu added this pull request to the merge queue Jul 24, 2026
Merged via the queue into main with commit 9c600ff Jul 24, 2026
15 checks passed
@Rayna-Yu
Rayna-Yu deleted the 241-add-auth-projects branch July 24, 2026 00:36
github-actions Bot added a commit that referenced this pull request Jul 24, 2026
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.

[Security] projects lambda: write/read routes have no authentication + mass-assignment

3 participants