Ronin
Ronin
PostsComments
CCConvex Community
Created by Ronin on 12/11/2024 in #support-community
Race condition error upon await ctx.auth.getUserIdentity();[revised post]
Okay, thanks Tom, let me take some time to find another example that more isolates our perceived race condition. I'll create a new support case if needed. But one general question: how should I handle the UserIdentity call? Should I throw an Error or return a null?
19 replies
CCConvex Community
Created by Ronin on 12/11/2024 in #support-community
Race condition error upon await ctx.auth.getUserIdentity();[revised post]
didn't adding a delay gave time to return authentication verification beacause before that even if someone is loggedin the auth was not working for them?
19 replies
CCConvex Community
Created by Ronin on 12/11/2024 in #support-community
Race condition error upon await ctx.auth.getUserIdentity();[revised post]
I did that when I went with the approach to check for space first, are you suggesting going back to the way the function was written before and change the return null instead of throwing an error
19 replies
CCConvex Community
Created by Ronin on 12/10/2024 in #support-community
[deprecated] Race condition error upon await ctx.auth.getUserIdentity();
https://discord.com/channels/1019350475847499849/1316445807066746942
11 replies
CCConvex Community
Created by Ronin on 12/10/2024 in #support-community
[deprecated] Race condition error upon await ctx.auth.getUserIdentity();
@ballingt here is the link of the new support post with better explaination of the problem with more context
11 replies
CCConvex Community
Created by Ronin on 12/11/2024 in #support-community
Race condition error upon await ctx.auth.getUserIdentity();[revised post]
To fix this I had to first check for the space instead of checking for authentication(we are using clerk for authentication vendor): 
export const getById = query({
    args: {
        id: v.id("spaces")
    },
    handler: async(ctx, args) => {
        const space = await ctx.db.get(args.id);
        
        if (!space) {
            return null;
        }

        const identity = await ctx.auth.getUserIdentity();
        
        // If not authenticated, return null
        if (!identity) {
            return null;
        }

        const user = await ctx.db
            .query("users")
            .withIndex("by_token", (q) => q.eq("tokenIdentifier", identity.tokenIdentifier))
            .unique();

        if (!user) {
            return null;
        }

        const member = await ctx.db
            .query("spaceMembers")
            .withIndex("by_space_id_user_id", (q) => 
                q.eq("spaceId", args.id).eq("userId", user._id)   
            )
            .unique();

        if (!member) {
            return null;
        }

        return space;
    }
});
export const getById = query({
    args: {
        id: v.id("spaces")
    },
    handler: async(ctx, args) => {
        const space = await ctx.db.get(args.id);
        
        if (!space) {
            return null;
        }

        const identity = await ctx.auth.getUserIdentity();
        
        // If not authenticated, return null
        if (!identity) {
            return null;
        }

        const user = await ctx.db
            .query("users")
            .withIndex("by_token", (q) => q.eq("tokenIdentifier", identity.tokenIdentifier))
            .unique();

        if (!user) {
            return null;
        }

        const member = await ctx.db
            .query("spaceMembers")
            .withIndex("by_space_id_user_id", (q) => 
                q.eq("spaceId", args.id).eq("userId", user._id)   
            )
            .unique();

        if (!member) {
            return null;
        }

        return space;
    }
});
Expected Behavior : The function was expected to return space details without failing at the point 
ctx.auth.getUserIdentity();
        if(!identity){
            throw new Error("Not authenticated");
        }
ctx.auth.getUserIdentity();
        if(!identity){
            throw new Error("Not authenticated");
        }
How this error can be reproduced : The error can be reproduced just by first trying to check for authentication before any other operation which triggers the race condition. The problem does not always occur frequently. We have worked with cases where User A would never see the issue and User B would always see the issue. The file in which this problem came is named spaces.ts. I have attached the file with the code
19 replies
CCConvex Community
Created by Ronin on 12/10/2024 in #support-community
[deprecated] Race condition error upon await ctx.auth.getUserIdentity();
The race condition I encountered was likely due to the way the ctx.db.get(args.id) call was being made. By first fetching the space data and then checking for the user and membership, I was able to avoid the race condition. However, I believe there may be a more optimal solution to this problem. Could you please provide guidance on the best way to structure this query to avoid race conditions and ensure the most efficient data retrieval?
11 replies
CCConvex Community
Created by Ronin on 12/10/2024 in #support-community
[deprecated] Race condition error upon await ctx.auth.getUserIdentity();
export const getById = query({
    args: {
        id: v.id("spaces")
    },
    handler: async(ctx, args) => {
        const space = await ctx.db.get(args.id);
        
        if (!space) {
            return null;
        }

        const identity = await ctx.auth.getUserIdentity();
        
        // If not authenticated, return null
        if (!identity) {
            return null;
        }

        const user = await ctx.db
            .query("users")
            .withIndex("by_token", (q) => q.eq("tokenIdentifier", identity.tokenIdentifier))
            .unique();

        if (!user) {
            return null;
        }

        const member = await ctx.db
            .query("spaceMembers")
            .withIndex("by_space_id_user_id", (q) => 
                q.eq("spaceId", args.id).eq("userId", user._id)   
            )
            .unique();

        if (!member) {
            return null;
        }

        return space;
    }
});
export const getById = query({
    args: {
        id: v.id("spaces")
    },
    handler: async(ctx, args) => {
        const space = await ctx.db.get(args.id);
        
        if (!space) {
            return null;
        }

        const identity = await ctx.auth.getUserIdentity();
        
        // If not authenticated, return null
        if (!identity) {
            return null;
        }

        const user = await ctx.db
            .query("users")
            .withIndex("by_token", (q) => q.eq("tokenIdentifier", identity.tokenIdentifier))
            .unique();

        if (!user) {
            return null;
        }

        const member = await ctx.db
            .query("spaceMembers")
            .withIndex("by_space_id_user_id", (q) => 
                q.eq("spaceId", args.id).eq("userId", user._id)   
            )
            .unique();

        if (!member) {
            return null;
        }

        return space;
    }
});
11 replies
CCConvex Community
Created by Ronin on 12/10/2024 in #support-community
[deprecated] Race condition error upon await ctx.auth.getUserIdentity();
This code resulted in a race condition error, as shown in the attached screenshot. To mitigate the issue, I made the following changes:
11 replies
CCConvex Community
Created by Ronin on 11/18/2024 in #support-community
Cannot access the "page" return type for pagination
thanks for replying
8 replies
CCConvex Community
Created by Ronin on 11/18/2024 in #support-community
Cannot access the "page" return type for pagination
now it's fixed
8 replies
CCConvex Community
Created by Ronin on 11/18/2024 in #support-community
Cannot access the "page" return type for pagination
though I found a way to fix it
8 replies
CCConvex Community
Created by Ronin on 11/18/2024 in #support-community
Cannot access the "page" return type for pagination
yeah I read about it
8 replies
CCConvex Community
Created by Ronin on 11/18/2024 in #support-community
Cannot access the "page" return type for pagination
No description
8 replies
CCConvex Community
Created by Ronin on 9/28/2024 in #support-community
Convex api not working and causing problem in user validations from the convex tables
Please let me know if you require any further details to help diagnose the problem.
8 replies
CCConvex Community
Created by Ronin on 9/28/2024 in #support-community
Convex api not working and causing problem in user validations from the convex tables
Hey @sshader, I recently deployed our website preview on Vercel, and while users in India are able to interact with the site without any issues, my colleague in the United States has been experiencing an authentication error. I would appreciate your assistance in resolving this issue, as it seems to be location-specific.
8 replies
CCConvex Community
Created by Ronin on 9/28/2024 in #support-community
Convex api not working and causing problem in user validations from the convex tables
Yeah I might have some args missing in my frontend but now that @sshader gave some hints to solve the issue I'll start with that approach, but thanks for helping 🙂
8 replies
CCConvex Community
Created by Ronin on 9/28/2024 in #support-community
Convex api not working and causing problem in user validations from the convex tables
Okay thanks, that was very helpful
8 replies