fix: user existence checks in session object

[ShubhamPalriwala]

What does this PR do?

Sentry runtime errors included these 2 pages where the id of an undefined object was being fetched resulting in a runtime error. This PR handles them both.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist

Required

• Filled out the "How to test" section in this PR
• Read How we Code at Formbricks
• Self-reviewed my own code
• Commented on my code in hard-to-understand bits
• Ran pnpm build
• Checked for warnings, there are none
• Removed all console.logs
• Merged the latest changes from main onto my branch with git pull origin main
• My changes don't cause any responsiveness issues
• First PR at Formbricks? Please sign the CLA! Without it we wont be able to merge it 🙏

Appreciated

• If a UI change was made: Added a screen recording or screenshots to this PR
• Updated the Formbricks Docs if changes were necessary

[vercel]

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

2 Ignored Deployments

Name	Status	Preview	Updated (UTC)
formbricks-cloud	⬜️ Ignored (Inspect)		Nov 7, 2023 6:37am
formbricks-com	⬜️ Ignored (Inspect)		Nov 7, 2023 6:37am

[github-actions]

Thank you for following the naming conventions for pull request titles! 🙏

[review-agent-prime]

apps/web/app/(app)/environments/[environmentId]/layout.tsx

It's a good practice to check if the session and session.user exist before using it. This can prevent potential null reference exceptions which can lead to application crashes.
Create Issue

    const session = await getServerSession(authOptions);
    if (!session || !session.user) {
      return redirect(`/auth/login`);
    }
    const hasAccess = await hasUserEnvironmentAccess(session.user.id, params.environmentId);

apps/web/app/(app)/environments/[environmentId]/settings/profile/page.tsx

It's a good practice to check if the session and session.user exist before using it. This can prevent potential null reference exceptions which can lead to application crashes.
Create Issue

    const session = await getServerSession(authOptions);
    const profile = session && session.user ? await getProfile(session.user.id) : null;

[review-agent-prime]

Added a check to ensure that the session and session.user exist before using it. This prevents potential null reference exceptions which can lead to application crashes.

Suggested change

	const session = await getServerSession(authOptions);
	const profile = session ? await getProfile(session.user.id) : null;
	const profile = session && session.user ? await getProfile(session.user.id) : null;
	const session = await getServerSession(authOptions);
	const profile = session && session.user ? await getProfile(session.user.id) : null;

[mattinannt]

😊🙏