asyavee/medusa-store
1
0 Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

fix: Use correct config for auth methods per actor (#7515)

Browse Source
This commit is contained in:
Stevche Radevski authored and GitHub committed 2024-05-29 10:42:53 +02:00
1 parent 51ea166df2
commit 8e66e10995
3 files changed
+9 -7
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

packages/core/types/src/common/config-module.ts
+1 -1
View File
@@ -735,7 +735,7 @@ export type ProjectConfigOptions = {
* } * }
* ``` * ```
*/ */
authMethodsPerActor: Record<string, string[]> authMethodsPerActor?: Record<string, string[]>
} }
} }
packages/medusa/src/api/auth/[actor_type]/[auth_provider]/callback/route.ts
+4 -3
View File
@@ -14,10 +14,11 @@ export const GET = async (req: MedusaRequest, res: MedusaResponse) => {
ContainerRegistrationKeys.CONFIG_MODULE ContainerRegistrationKeys.CONFIG_MODULE
) )
const authMethods = (config.projectConfig?.http as any)?.authMethods ?? {} const authMethodsPerActor =
config.projectConfig?.http?.authMethodsPerActor ?? {}
// Not having the config defined would allow for all auth providers for the particular actor. // Not having the config defined would allow for all auth providers for the particular actor.
if (authMethods[actor_type]) { if (authMethodsPerActor[actor_type]) {
if (!authMethods[actor_type].includes(auth_provider)) { if (!authMethodsPerActor[actor_type].includes(auth_provider)) {
throw new MedusaError( throw new MedusaError(
MedusaError.Types.NOT_ALLOWED, MedusaError.Types.NOT_ALLOWED,
`The actor type ${actor_type} is not allowed to use the auth provider ${auth_provider}` `The actor type ${actor_type} is not allowed to use the auth provider ${auth_provider}`
packages/medusa/src/api/auth/[actor_type]/[auth_provider]/route.ts
+4 -3
View File
@@ -14,10 +14,11 @@ export const GET = async (req: MedusaRequest, res: MedusaResponse) => {
ContainerRegistrationKeys.CONFIG_MODULE ContainerRegistrationKeys.CONFIG_MODULE
) )
const authMethods = (config.projectConfig?.http as any)?.authMethods ?? {} const authMethodsPerActor =
config.projectConfig?.http?.authMethodsPerActor ?? {}
// Not having the config defined would allow for all auth providers for the particular actor. // Not having the config defined would allow for all auth providers for the particular actor.
if (authMethods[actor_type]) { if (authMethodsPerActor[actor_type]) {
if (!authMethods[actor_type].includes(auth_provider)) { if (!authMethodsPerActor[actor_type].includes(auth_provider)) {
throw new MedusaError( throw new MedusaError(
MedusaError.Types.NOT_ALLOWED, MedusaError.Types.NOT_ALLOWED,
`The actor type ${actor_type} is not allowed to use the auth provider ${auth_provider}` `The actor type ${actor_type} is not allowed to use the auth provider ${auth_provider}`