Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated declaration merging interface, made serviceKey optional, exported plugin opts interface #38

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

vehm
Copy link

@vehm vehm commented Oct 8, 2024

Did a few things:

  • Updated the declaration merging FastifyRequest interface to include supabaseClient.
  • Made serviceKey optional to allow for users to register the plugin without creating a potentially dangerous client (using service_role) that bypasses all RLS policies for Supabase.
  • Exported the FastifySupabasePluginOpts from index.ts

Fixes #28
Fixes #34

@vehm vehm changed the title Updated declaration merging interface, made serviceKey optional Updated declaration merging interface, made serviceKey optional, exported plugin opts interface Oct 8, 2024
@vehm
Copy link
Author

vehm commented Oct 28, 2024

Updated Fastify decoration reference type for v5 support:
Decorating Request/Reply with a reference type (Array, Object) is now prohibited as this reference is shared amongst all requests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant