Skip to content

Conversation

@matteyu
Copy link
Contributor

@matteyu matteyu commented Mar 26, 2025

Jira: https://communitylabs.atlassian.net/browse/ARC-1133?atlOrigin=eyJpIjoiN2U0M2UxMjQ0OTY0NGQ3YTkyYmRhODQ2YTI5MzlkM2YiLCJwIjoiaiJ9

  • Add row level security to tables via migration files
  • Added a condition for an authenticated prisma client

@vercel
Copy link

vercel bot commented Mar 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
embed-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 5, 2025 5:09am

@matteyu matteyu changed the title feat: add row level security [WIP] feat: add row level security Mar 26, 2025
@matteyu matteyu changed the title [WIP] feat: add row level security feat: add row level security Apr 2, 2025
@matteyu matteyu requested review from Danziger and pawanpaudel93 and removed request for pawanpaudel93 April 2, 2025 15:27
@matteyu matteyu changed the base branch from arc-937/auth-passkeys to development April 2, 2025 15:27
@matteyu matteyu changed the base branch from development to arc-937/auth-passkeys April 2, 2025 15:28
@matteyu matteyu force-pushed the arc-1133/row-level-security branch from 1cb83fa to 5de4eee Compare April 3, 2025 05:10
@matteyu matteyu changed the base branch from arc-937/auth-passkeys to development April 3, 2025 05:10
@Danziger
Copy link
Contributor

Danziger commented Apr 3, 2025

@7i7o No need to update the embed-api SDK on the app for this one. After one more review we can merge it.

Copy link
Contributor

@Danziger Danziger left a comment

Choose a reason for hiding this comment

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

Some policies can be refined but this can be improved on a new PR. Also, some discussions is needed around plans and usage.

Copy link
Contributor

@pawanpaudel93 pawanpaudel93 left a comment

Choose a reason for hiding this comment

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

uTACK. This needs to be tested with our DB so it works without causing any issues.

@matteyu
Copy link
Contributor Author

matteyu commented Apr 29, 2025

Complete flow with RLS: https://drive.google.com/file/d/1ZyT0P7026hgl4Z6-VNqP9hbchGOTRWoH/view?usp=drive_link

  • login tested
  • wallet creation tested
  • wallet backup tested
  • wallet restore tested

@matteyu matteyu changed the base branch from development to arc-937/auth-passkeys May 4, 2025 22:10
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.

4 participants