Skip to content

Commit 260fa5f

Browse files
authored
Merge branch 'main' into main
2 parents 911022f + d953ce0 commit 260fa5f

127 files changed

Lines changed: 4540 additions & 1821 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

backend/.env.dist.local

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,5 +165,5 @@ CROWD_GITHUB_IS_SNOWFLAKE_ENABLED=false
165165
CROWD_TINYBIRD_BASE_URL=http://localhost:7181/
166166

167167
# Auth0
168-
CROWD_AUTH0_ISSUER_BASE_URL=
168+
CROWD_AUTH0_ISSUER_BASE_URLS=
169169
CROWD_AUTH0_AUDIENCE=

backend/config/custom-environment-variables.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@
154154
"auth0": {
155155
"clientId": "CROWD_AUTH0_CLIENT_ID",
156156
"jwks": "CROWD_AUTH0_JWKS",
157-
"issuerBaseURL": "CROWD_AUTH0_ISSUER_BASE_URL",
157+
"issuerBaseURLs": "CROWD_AUTH0_ISSUER_BASE_URLS",
158158
"audience": "CROWD_AUTH0_AUDIENCE"
159159
},
160160
"sso": {

backend/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@
128128
"pg": "^8.7.3",
129129
"pg-promise": "^11.4.3",
130130
"sanitize-html": "^2.7.1",
131-
"sequelize": "6.21.2",
131+
"sequelize": "6.37.8",
132132
"sequelize-cli-typescript": "^3.2.0-c",
133133
"slack-block-builder": "^2.7.2",
134134
"socket.io": "^4.5.4",

backend/src/api/index.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import * as http from 'http'
77
import os from 'os'
88
import { QueryTypes } from 'sequelize'
99

10+
import { BadRequestError } from '@crowd/common'
1011
import { getDbConnection } from '@crowd/data-access-layer/src/database'
1112
import { getServiceLogger } from '@crowd/logging'
1213
import { getOpensearchClient } from '@crowd/opensearch'
@@ -147,6 +148,14 @@ setImmediate(async () => {
147148

148149
app.use(bodyParser.urlencoded({ limit: '5mb', extended: true }))
149150

151+
app.use((err: any, req: any, res: any, next: any) => {
152+
if (err.type === 'entity.parse.failed') {
153+
next(new BadRequestError('Invalid JSON body'))
154+
return
155+
}
156+
next(err)
157+
})
158+
150159
app.use((req, res, next) => {
151160
// @ts-ignore
152161
req.userData = {

backend/src/api/member/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ export default (app) => {
99
app.put(`/member/:id`, safeWrap(require('./memberUpdate').default))
1010
app.delete(`/member`, safeWrap(require('./memberDestroy').default))
1111
app.post(`/member/autocomplete`, safeWrap(require('./memberAutocomplete').default))
12-
app.get(`/member/active`, safeWrap(require('./memberActiveList').default))
1312
app.get(`/member/bot-suggestions`, safeWrap(require('./memberBotSuggestionsList').default))
1413

1514
app.get(`/member/:id`, safeWrap(require('./memberFind').default))

backend/src/api/member/memberActiveList.ts

Lines changed: 0 additions & 76 deletions
This file was deleted.

backend/src/api/organization/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ export default (app) => {
66
app.put(`/organization/:id`, safeWrap(require('./organizationUpdate').default))
77
app.delete(`/organization`, safeWrap(require('./organizationDestroy').default))
88
app.post(`/organization/autocomplete`, safeWrap(require('./organizationAutocomplete').default))
9-
app.get(`/organization/active`, safeWrap(require('./organizationActiveList').default))
109
app.get(`/organization/:id`, safeWrap(require('./organizationFind').default))
1110

1211
app.put(`/organization/:organizationId/merge`, safeWrap(require('./organizationMerge').default))

backend/src/api/organization/organizationActiveList.ts

Lines changed: 0 additions & 52 deletions
This file was deleted.

backend/src/api/public/index.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,12 @@
11
import { Router } from 'express'
22

3-
import { AUTH0_CONFIG } from '../../conf'
4-
53
import { errorHandler } from './middlewares/errorHandler'
6-
import { oauth2Middleware } from './middlewares/oauth2Middleware'
7-
import { staticApiKeyMiddleware } from './middlewares/staticApiKeyMiddleware'
84
import { v1Router } from './v1'
9-
import { devStatsRouter } from './v1/dev-stats'
105

116
export function publicRouter(): Router {
127
const router = Router()
138

14-
router.use('/v1/dev-stats', staticApiKeyMiddleware(), devStatsRouter())
15-
router.use('/v1', oauth2Middleware(AUTH0_CONFIG), v1Router())
9+
router.use('/v1', v1Router())
1610
router.use(errorHandler)
1711

1812
return router

backend/src/api/public/middlewares/errorHandler.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,13 @@ export const errorHandler: ErrorRequestHandler = (
3535
}
3636

3737
req.log.error(
38-
{ error, url: req.url, method: req.method, query: req.query, body: req.body },
38+
{
39+
error: { name: error?.name, message: error?.message, stack: error?.stack },
40+
url: req.url,
41+
method: req.method,
42+
query: req.query,
43+
body: req.body,
44+
},
3945
'Unhandled error in public API',
4046
)
4147

0 commit comments

Comments
 (0)