Skip to content

Commit

Permalink
fix(viewer): 🐛 Forwarded host check attempt
Browse files Browse the repository at this point in the history
  • Loading branch information
baptisteArno committed May 23, 2022
1 parent a2dd26b commit c168b67
Showing 1 changed file with 20 additions and 8 deletions.
28 changes: 20 additions & 8 deletions apps/viewer/pages/[[...publicId]].tsx
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { IncomingMessage } from 'http'
import { NotFoundPage } from 'layouts/NotFoundPage'
import { PublicTypebot } from 'models'
import { GetServerSideProps, GetServerSidePropsContext } from 'next'
Expand All @@ -11,15 +12,19 @@ export const getServerSideProps: GetServerSideProps = async (
let typebot: Omit<PublicTypebot, 'createdAt' | 'updatedAt'> | null
const isIE = /MSIE|Trident/.test(context.req.headers['user-agent'] ?? '')
const pathname = context.resolvedUrl.split('?')[0]
const { host, forwardedHost } = getHost(context.req)
try {
if (!context.req.headers.host) return { props: {} }
if (!host) return { props: {} }
const viewerUrls = (process.env.NEXT_PUBLIC_VIEWER_URL ?? '').split(',')
const isMatchingViewerUrl = viewerUrls.some((url) =>
(context.req.headers.host ?? '')
.split(':')[0]
.includes(url.split('//')[1].split(':')[0])
const isMatchingViewerUrl = viewerUrls.some(
(url) =>
host.split(':')[0].includes(url.split('//')[1].split(':')[0]) ||
(forwardedHost &&
forwardedHost
.split(':')[0]
.includes(url.split('//')[1].split(':')[0]))
)
const customDomain = `${context.req.headers.host}${
const customDomain = `${forwardedHost ?? host}${
pathname === '/' ? '' : pathname
}`
typebot = isMatchingViewerUrl
Expand All @@ -35,7 +40,7 @@ export const getServerSideProps: GetServerSideProps = async (
props: {
typebot,
isIE,
url: `https://${context.req.headers.host}${pathname}`,
url: `https://${forwardedHost ?? host}${pathname}`,
},
}
} catch (err) {
Expand All @@ -44,7 +49,7 @@ export const getServerSideProps: GetServerSideProps = async (
return {
props: {
isIE,
url: `https://${context.req.headers.host}${pathname}`,
url: `https://${forwardedHost ?? host}${pathname}`,
},
}
}
Expand All @@ -66,6 +71,13 @@ const getTypebotFromCustomDomain = async (customDomain: string) => {
return omit(typebot as unknown as PublicTypebot, 'createdAt', 'updatedAt')
}

const getHost = (
req?: IncomingMessage
): { host?: string; forwardedHost?: string } => ({
host: req?.headers ? req.headers.host : window.location.host,
forwardedHost: req?.headers['x-forwarded-host'] as string | undefined,
})

const App = ({ typebot, ...props }: TypebotPageProps) =>
isDefined(typebot) ? (
<TypebotPage typebot={typebot} {...props} />
Expand Down

3 comments on commit c168b67

@vercel
Copy link

@vercel vercel bot commented on c168b67 May 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vercel
Copy link

@vercel vercel bot commented on c168b67 May 23, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vercel
Copy link

@vercel vercel bot commented on c168b67 May 23, 2022

Please sign in to comment.