Skip to content

Commit 06749a2

Browse files
authored
Merge pull request #1207 from navikt/bygger-ansatt-domene
Bygger ansatt domene
2 parents 7edf624 + 56e5dff commit 06749a2

File tree

7 files changed

+66
-5
lines changed

7 files changed

+66
-5
lines changed

Diff for: .nais/bygger/preprod-alt.yaml

+5-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
app-name: skjemabygging-preprod-alt
22
ingresses:
33
- "https://skjemabygging-preprod-alt.intern.dev.nav.no"
4-
fyllut-base-url: https://fyllut-preprod-alt.intern.dev.nav.no/fyllut
4+
- "https://skjemabygging-preprod-alt.ansatt.dev.nav.no"
5+
fyllut-base-url: http://skjemautfylling-preprod-alt/fyllut
56
enabled-features: translations,diff
67
publish-repo: skjemautfylling-formio
78
publish-repo-owner: navikt
@@ -20,8 +21,10 @@ accessPolicy:
2021
- application: formio-enterprise-server
2122
namespace: skjemadigitalisering
2223
cluster: dev-gcp
24+
- application: skjemautfylling-preprod-alt
25+
namespace: skjemadigitalisering
26+
cluster: dev-gcp
2327
external:
24-
- host: fyllut-preprod-alt.intern.dev.nav.no
2528
- host: skjemabygging-proxy.dev-fss-pub.nais.io
2629
secrets:
2730
- formio-server-access-dev

Diff for: .nais/bygger/preprod.yaml

+5-2
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,10 @@ app-name: skjemabygging-preprod
22
env-name: dev-gcp
33
ingresses:
44
- "https://skjemabygging-preprod.intern.dev.nav.no"
5+
- "https://skjemabygging-preprod.ansatt.dev.nav.no"
56
logs-url: https://logs.adeo.no/app/r/s/lvnlu
67
special-type-to-use-in-alertmanager-config: "fyllut-sendinn-alerts-dev"
7-
fyllut-base-url: https://fyllut-preprod.intern.dev.nav.no/fyllut
8+
fyllut-base-url: http://skjemautfylling-preprod/fyllut
89
enabled-features: translations,diff
910
publish-repo: skjemautfylling-formio
1011
publish-repo-owner: navikt
@@ -23,8 +24,10 @@ accessPolicy:
2324
- application: formio-enterprise-server
2425
namespace: skjemadigitalisering
2526
cluster: dev-gcp
27+
- application: skjemautfylling-preprod
28+
namespace: skjemadigitalisering
29+
cluster: dev-gcp
2630
external:
27-
- host: fyllut-preprod.intern.dev.nav.no
2831
- host: skjemabygging-proxy.dev-fss-pub.nais.io
2932
secrets:
3033
- formio-server-access-dev

Diff for: .nais/fyllut/preprod-alt.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@ accessPolicy:
2525
rules:
2626
- application: send-inn-frontend-alt
2727
namespace: team-soknad
28+
- application: skjemabygging-preprod-alt
29+
namespace: skjemadigitalisering
2830
outbound:
2931
rules:
3032
- application: skjemabygging-proxy

Diff for: .nais/fyllut/preprod.yaml

+2
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,8 @@ accessPolicy:
2525
rules:
2626
- application: send-inn-frontend
2727
namespace: team-soknad
28+
- application: skjemabygging-preprod
29+
namespace: skjemadigitalisering
2830
outbound:
2931
rules:
3032
- application: skjemabygging-proxy

Diff for: packages/bygger-backend/src/routers/api/config.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ const config = (req: ByggerRequest, res: Response) => {
1111
}
1212
res.json({
1313
formioProjectUrl: `${getByggerUrl(req)}/${appConfig.formio.projectName}`,
14-
fyllutBaseUrl: appConfig.fyllut.baseUrl,
14+
fyllutBaseUrl: `${getByggerUrl(req)}/fyllut`,
1515
pusherCluster: appConfig.pusher.cluster,
1616
pusherKey: appConfig.pusher.key,
1717
isDevelopment: appConfig.isDevelopment,
+49
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,49 @@
1+
import express from 'express';
2+
import httpProxy from 'http-proxy';
3+
import config from '../../config';
4+
import { logger } from '../../logging/logger';
5+
6+
const fyllutBaseUrl = config.fyllut.baseUrl;
7+
const proxy = httpProxy.createProxyServer({});
8+
proxy.on('proxyReq', (proxyReq) => {
9+
proxyReq.removeHeader('authorization');
10+
});
11+
12+
const fyllutProxyRouter = express.Router();
13+
14+
function logProxyError(proxyMessage: string, err: any, method: string, url: string) {
15+
const { message, stack, ...errDetails } = err;
16+
logger.error(`${proxyMessage}: ${message}`, {
17+
method,
18+
url,
19+
fyllutBaseUrl,
20+
stack,
21+
errDetails,
22+
});
23+
}
24+
25+
fyllutProxyRouter.all('*', async (req, res, next) => {
26+
const { method, url } = req;
27+
try {
28+
logger.info(`Fyllut proxy ${method} ${url}`, { fyllutBaseUrl });
29+
proxy.web(
30+
req,
31+
res,
32+
{
33+
target: fyllutBaseUrl,
34+
changeOrigin: true,
35+
proxyTimeout: 60000,
36+
timeout: 60000,
37+
},
38+
(err) => {
39+
logProxyError('Fyllut proxy - error callback', err, method, url);
40+
next(err);
41+
},
42+
);
43+
} catch (err: any) {
44+
logProxyError('Fyllut proxy - error was thrown', err, method, url);
45+
next(err);
46+
}
47+
});
48+
49+
export default fyllutProxyRouter;

Diff for: packages/bygger-backend/src/server.js

+2
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,15 @@ import authHandler from './middleware/authHandler';
66
import { fsAccessRateLimiter } from './middleware/ratelimit';
77
import apiRouter from './routers/api';
88
import formioProxyRouter from './routers/formio-proxy';
9+
import fyllutProxyRouter from './routers/fyllut-proxy';
910
import internalRouter from './routers/internal';
1011
import notificationsRouter from './routers/notifications';
1112
import './util/errorToJson';
1213

1314
const app = express();
1415

1516
app.use(`/${config.formio.projectName}`, formioProxyRouter);
17+
app.use('/fyllut', fyllutProxyRouter);
1618
app.use(express.json({ limit: '50mb' }));
1719
app.use(express.urlencoded({ extended: true, limit: '50mb' }));
1820
app.use(correlator());

0 commit comments

Comments
 (0)