Skip to content

Commit 1c7a6b1

Browse files
committed
refactor(ts): refactor src/config imports
1 parent 33c27a1 commit 1c7a6b1

File tree

7 files changed

+15
-15
lines changed

7 files changed

+15
-15
lines changed

index.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
import yargs from 'yargs';
44
import { hideBin } from 'yargs/helpers';
55
import * as fs from 'fs';
6-
import config from './src/config/file';
6+
import { configFile, setConfigFile, validate } from './src/config/file';
77
import proxy from './src/proxy';
88
import service from './src/service';
99

@@ -28,22 +28,22 @@ const argv = yargs(hideBin(process.argv))
2828
.strict()
2929
.parseSync();
3030

31-
config.configFile = argv.c ? argv.c : undefined;
31+
setConfigFile(argv.c as string || "");
3232

3333
if (argv.v) {
34-
if (!fs.existsSync(config.configFile as string)) {
34+
if (!fs.existsSync(configFile)) {
3535
console.error(
36-
`Config file ${config.configFile} doesn't exist, nothing to validate! Did you forget -c/--config?`,
36+
`Config file ${configFile} doesn't exist, nothing to validate! Did you forget -c/--config?`,
3737
);
3838
process.exit(1);
3939
}
4040

41-
config.validate();
42-
console.log(`${config.configFile} is valid`);
41+
validate();
42+
console.log(`${configFile} is valid`);
4343
process.exit(0);
4444
}
4545

46-
config.validate();
46+
validate();
4747

4848
proxy.start();
4949
service.start();

packages/git-proxy-cli/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,8 @@ const util = require('util');
77

88
const GIT_PROXY_COOKIE_FILE = 'git-proxy-cookie';
99
// GitProxy UI HOST and PORT (configurable via environment variable)
10-
const { GIT_PROXY_UI_HOST: uiHost = 'http://localhost' } = process.env;
11-
const { GIT_PROXY_UI_PORT: uiPort } = require('@finos/git-proxy/src/config/env').Vars;
10+
const { GIT_PROXY_UI_HOST: uiHost, GIT_PROXY_UI_PORT: uiPort } =
11+
require('@finos/git-proxy/src/config/env').serverConfig;
1212
const baseUrl = `${uiHost}:${uiPort}`;
1313

1414
axios.defaults.timeout = 30000;

src/db/file/repo.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import fs from 'fs';
22
import Datastore from '@seald-io/nedb'
3-
import { Action } from '../../proxy/actions/Action';
43
import { Repo } from '../types';
54

65
if (!fs.existsSync('./.data')) fs.mkdirSync('./.data');

src/proxy/index.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ const config = require('../config');
99
const db = require('../db');
1010
const { PluginLoader } = require('../plugin');
1111
const chain = require('./chain');
12-
const { GIT_PROXY_SERVER_PORT: proxyHttpPort } = require('../config/env').Vars;
13-
const { GIT_PROXY_HTTPS_SERVER_PORT: proxyHttpsPort } = require('../config/env').Vars;
12+
const { GIT_PROXY_SERVER_PORT: proxyHttpPort, GIT_PROXY_HTTPS_SERVER_PORT: proxyHttpsPort } =
13+
require('../config/env').serverConfig;
1414

1515
const options = {
1616
inflate: true,

src/service/index.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ const limiter = rateLimit({
1414
max: 100, // limit each IP to 100 requests per windowMs
1515
});
1616

17-
const { GIT_PROXY_UI_PORT: uiPort } = require('../config/env').Vars;
17+
const { GIT_PROXY_UI_PORT: uiPort } = require('../config/env').serverConfig;
1818

1919
const _httpServer = http.createServer(app);
2020

src/service/urls.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const { GIT_PROXY_SERVER_PORT: PROXY_HTTP_PORT, GIT_PROXY_UI_PORT: UI_PORT } =
2-
require('../config/env').Vars;
2+
require('../config/env').serverConfig;
33
const config = require('../config');
44

55
module.exports = {

tsconfig.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@
1212
"isolatedModules": true,
1313
"module": "CommonJS",
1414
"esModuleInterop": true,
15-
"allowSyntheticDefaultImports": true
15+
"allowSyntheticDefaultImports": true,
16+
"resolveJsonModule": true
1617
},
1718
"include": ["src"]
1819
}

0 commit comments

Comments
 (0)