James! Comments - Answer Overflow

James!

•Created by Tarun kumar on 2/25/2025 in #help

Magic Link Type Error

https://github.com/better-auth/better-auth/pull/1838 - I created a fix for this and it has been merged. If you install the latest beta tag, you will be able to get this

10 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

I have opened a PR for this: https://github.com/better-auth/better-auth/pull/1838

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

Yeah it is. Going to have another look today

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

Okay, I found the issue:

export function getIp(
    req: Request | Headers,
    options: BetterAuthOptions,
): string | null {
    if (options.advanced?.ipAddress?.disableIpTracking) {
        return null;
    }
    const testIP = "127.0.0.1";
    if (isTest) {
        return testIP;
    }
    const ipHeaders = options.advanced?.ipAddress?.ipAddressHeaders;
    const keys = ipHeaders || [
        "x-client-ip",
        "x-forwarded-for",
        "cf-connecting-ip",
        "fastly-client-ip",
        "x-real-ip",
        "x-cluster-client-ip",
        "x-forwarded",
        "forwarded-for",
        "forwarded",
    ];
    const headers = req instanceof Request ? req.headers : req;
    console.log("headers in getReqIp", headers);
    for (const key of keys) {
        const value = headers.get?.(key); <~ I addewd the ?. here
        if (typeof value === "string") {
            const ip = value.split(",")[0].trim();
            if (ip) return ip;
        }
    }
    return null;
}

export function getIp(
    req: Request | Headers,
    options: BetterAuthOptions,
): string | null {
    if (options.advanced?.ipAddress?.disableIpTracking) {
        return null;
    }
    const testIP = "127.0.0.1";
    if (isTest) {
        return testIP;
    }
    const ipHeaders = options.advanced?.ipAddress?.ipAddressHeaders;
    const keys = ipHeaders || [
        "x-client-ip",
        "x-forwarded-for",
        "cf-connecting-ip",
        "fastly-client-ip",
        "x-real-ip",
        "x-cluster-client-ip",
        "x-forwarded",
        "forwarded-for",
        "forwarded",
    ];
    const headers = req instanceof Request ? req.headers : req;
    console.log("headers in getReqIp", headers);
    for (const key of keys) {
        const value = headers.get?.(key); <~ I addewd the ?. here
        if (typeof value === "string") {
            const ip = value.split(",")[0].trim();
            if (ip) return ip;
        }
    }
    return null;
}

The problem was that headers.get.key() was returning the .get is not a function. NOW I haven't figured what the ROOT cause of this is. I initially thought it was because in nextjs 15 you needed to use await headers() BUT maybe this is only app pages, and not API routes.

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

I believe this is the culprit here: basePath, <~ Here is the culprit

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

After looking through the code I have narrowde it down to the better-call implementation:

return async <OPT extends O, K extends keyof OPT, C extends InferContext<OPT[K]>>(
        path: K,
        ...options: HasRequired<C> extends true
            ? [
                    WithRequired<
                        BetterFetchOption<C["body"], C["query"], C["params"]>,
                        keyof RequiredOptionKeys<C>
                    >,
                ]
            : [BetterFetchOption<C["body"], C["query"], C["params"]>?]
    ): Promise<
        BetterFetchResponse<Awaited<ReturnType<OPT[K] extends Endpoint ? OPT[K] : never>>>
    > => {
        return (await fetch(path as string, {
            ...options[0],
        })) as any;
    };
};

return async <OPT extends O, K extends keyof OPT, C extends InferContext<OPT[K]>>(
        path: K,
        ...options: HasRequired<C> extends true
            ? [
                    WithRequired<
                        BetterFetchOption<C["body"], C["query"], C["params"]>,
                        keyof RequiredOptionKeys<C>
                    >,
                ]
            : [BetterFetchOption<C["body"], C["query"], C["params"]>?]
    ): Promise<
        BetterFetchResponse<Awaited<ReturnType<OPT[K] extends Endpoint ? OPT[K] : never>>>
    > => {
        return (await fetch(path as string, {
            ...options[0],
        })) as any;
    };
};

On the nextjs server, it is trying to fetch a relative path. As we can see in the better-auth -> src/api/index.ts in this code:

export const router = <C extends AuthContext, Option extends BetterAuthOptions>(
    ctx: C,
    options: Option,
) => {
    const { api, middlewares } = getEndpoints(ctx, options);
    // console.log("[BETTER_AUTH.router] options", JSON.stringify(options, null, 2));
    console.log(JSON.stringify(ctx, null, 2), 'ctx__')
    console.log(options, 'options')
    const basePath = new URL(ctx.baseURL).pathname;
    console.log("[BETTER_AUTH.router] basePath", basePath);
    console.log("[BETTER_AUTH.router] middlewares", JSON.stringify(middlewares, null, 2));

    return createRouter(api, {
        routerContext: ctx,
        openapi: {
            disabled: true,
        },
        basePath, <~ Here is the culprit

export const router = <C extends AuthContext, Option extends BetterAuthOptions>(
    ctx: C,
    options: Option,
) => {
    const { api, middlewares } = getEndpoints(ctx, options);
    // console.log("[BETTER_AUTH.router] options", JSON.stringify(options, null, 2));
    console.log(JSON.stringify(ctx, null, 2), 'ctx__')
    console.log(options, 'options')
    const basePath = new URL(ctx.baseURL).pathname;
    console.log("[BETTER_AUTH.router] basePath", basePath);
    console.log("[BETTER_AUTH.router] middlewares", JSON.stringify(middlewares, null, 2));

    return createRouter(api, {
        routerContext: ctx,
        openapi: {
            disabled: true,
        },
        basePath, <~ Here is the culprit

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

It looks like this might be a problem with opennext - I’m going to do some digging today. Perhaps there are some behaviours that are causing this. Perhaps requesting itself is playing up

37 replies

BABetter Auth

•Created by Tarun kumar on 2/25/2025 in #help

Magic Link Type Error

Are you hosting this on vercel or on your own opennext implementation? If opennext, I think there is an issue with some configuration or behaviour it has baked into it

10 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

No luck. Does better-auth have any logics around forwarding headers or cookies that might not be working well with a custom nextjs hosted solution via opennext

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

I feel like I have exhausted most of my known possible solutions to this issue I've just pushed this change to check your suggestion + added some extra logs. Apart from this - is there anything else you suggest I try if this doesn't yield any beneficial information? 🤔

export const POST = (req: Request) => {
    return getBetterAuth().handler(req)
        .then((res) => {
            console.info(res, 'POST /api/auth/[...all]')
            return res
        })
        .catch((err) => {
            console.error(err, 'ERR: POST /api/auth/[...all]')
            throw err
        })
        .finally(() => {
            console.info('FINALLY: POST /api/auth/[...all]')
        })
  };
  
  
  export const GET = (req: Request) => {
    return getBetterAuth().handler(req)
        .then((res) => {
            console.info(res, 'GET /api/auth/[...all]')
            return res
        })
        .catch((err) => {
            console.error(err, 'ERR: GET /api/auth/[...all]')
            throw err
        })
        .finally(() => {
            console.info('FINALLY: GET /api/auth/[...all]')
        })
  };

export const POST = (req: Request) => {
    return getBetterAuth().handler(req)
        .then((res) => {
            console.info(res, 'POST /api/auth/[...all]')
            return res
        })
        .catch((err) => {
            console.error(err, 'ERR: POST /api/auth/[...all]')
            throw err
        })
        .finally(() => {
            console.info('FINALLY: POST /api/auth/[...all]')
        })
  };
  
  
  export const GET = (req: Request) => {
    return getBetterAuth().handler(req)
        .then((res) => {
            console.info(res, 'GET /api/auth/[...all]')
            return res
        })
        .catch((err) => {
            console.error(err, 'ERR: GET /api/auth/[...all]')
            throw err
        })
        .finally(() => {
            console.info('FINALLY: GET /api/auth/[...all]')
        })
  };

Thanks for the tips BTW!

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

I will try this 👍 Do we need to set trustedOrigins - My assumption is that without setting these, those security configs just won't be used.

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

Nope it wasnt 😓 I thought it was the baseUrl being different to what my cloudfront URL was

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

🤞

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

Okay - about to test something which will be the classic embarrassing issue if it was true lol

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

Is there perhaps an "origin" policy I might be missing

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

I connected to my database locally from within VPN and it worked perfectly. I also confirmed the DB connection works as expected because I can see insertions to tables unrelated to better-auth.

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

- Db connection is okay - Tested fully on another feature unrelated to better auth. I'll focus around the DB side of things!

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

^ I then have my drizzle which maps the snake_case back to camelCase

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

I will triple confirm the schema in my remote DB @lonelyplanet but: my local works and it is using the same script that was applied in my hosted DB (see attached file) I really hope this is just some minor thing I have missed 😂 But it's hard to track down because the log isn't useful at all. I will also do the betterauth cli tonight

37 replies

BABetter Auth

•Created by James! on 3/12/2025 in #help

[magicLink - drizzleAdapter] The \"payload\" argument must be of type object. Received null

37 replies

Gaming

Programming