Skip to content

Commit a31cce4

Browse files
fix: dev server broke due to sentry package
1 parent 489f675 commit a31cce4

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

app/routes/_seo+/sitemap[.]xml.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,9 @@ import { getDomainUrl } from '#app/utils/misc.tsx'
44

55
export async function loader({ request, context }: LoaderFunctionArgs) {
66
const serverBuild = (await context.serverBuild) as { build: ServerBuild }
7+
8+
// TODO: This is typeerror is coming up since of the remix-run/server-runtime package. We might need to remove/update that one.
9+
// @ts-expect-error
710
return generateSitemap(request, serverBuild.build.routes, {
811
siteUrl: getDomainUrl(request),
912
headers: {

server/index.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
import crypto from 'node:crypto'
22
import { createRequestHandler } from '@react-router/express'
3-
import Sentry from '@sentry/react'
3+
import * as Sentry from '@sentry/react'
44
import { ip as ipAddress } from 'address'
55
import chalk from 'chalk'
66
import closeWithGrace from 'close-with-grace'
@@ -12,8 +12,6 @@ import helmet from 'helmet'
1212
import morgan from 'morgan'
1313
import { type ServerBuild } from 'react-router'
1414

15-
console.log('Starting server from index.ts...')
16-
1715
const MODE = process.env.NODE_ENV ?? 'development'
1816
const IS_PROD = MODE === 'production'
1917
const IS_DEV = MODE === 'development'
@@ -32,7 +30,7 @@ const viteDevServer = IS_PROD
3230
}),
3331
)
3432

35-
export const app = express()
33+
const app = express()
3634

3735
const getHost = (req: { get: (key: string) => string | undefined }) =>
3836
req.get('X-Forwarded-Host') ?? req.get('host') ?? ''
@@ -67,7 +65,7 @@ app.get('*', (req, res, next) => {
6765

6866
app.use(compression())
6967

70-
// http://expressjs.com/en/advanced/best-practice-security.html#at-a-minimum-disable-x-powered-by-header
68+
// // http://expressjs.com/en/advanced/best-practice-security.html#at-a-minimum-disable-x-powered-by-header
7169
app.disable('x-powered-by')
7270

7371
if (viteDevServer) {

0 commit comments

Comments
 (0)