Skip to content

Commit

Permalink
feat: fix up imports and make logger into a module
Browse files Browse the repository at this point in the history
  • Loading branch information
LewisBroadhurst committed Mar 29, 2024
1 parent dca1ed4 commit dccdc78
Show file tree
Hide file tree
Showing 14 changed files with 37 additions and 13 deletions.
5 changes: 3 additions & 2 deletions index.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
#!/usr/bin/env node
/* eslint-disable max-len */
const { logger } = require('./src/logging/index');

const argv = require('yargs/yargs')(process.argv.slice(2))
.usage('Usage: $0 [options]')
.options({
Expand Down Expand Up @@ -46,10 +48,9 @@ if (argv.v) {

const proxy = require('./src/proxy');
const service = require('./src/service');
const logger = require('./src/logging/logger');

proxy.start();
service.start();

module.exports.proxy = proxy;
module.exports.service = service;
module.exports.service = service;
23 changes: 23 additions & 0 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion scripts/prepare.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@

const { execSync } = require('child_process');
const { existsSync } = require('fs');
const logger = require('../src/logging/logger');
const { logger } = require('../src/logging/index');

// ===========
// File paths.
Expand Down
2 changes: 1 addition & 1 deletion src/config/index.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
const fs = require('fs');

const defaultSettings = require('../../proxy.config.json');
const logger = require('../../src/logging/logger');
const { logger } = require('../logging/index');
const userSettingsPath = require('./file').configFile;

let _userSettings = null;
Expand Down
2 changes: 1 addition & 1 deletion src/db/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ const passwordHash = require('password-hash');
const validator = require('email-validator');

const config = require('../config');
const logger = require('../../src/logging/logger');
const { logger } = require('../logging/index');

if (config.getDatabase().type === 'fs') {
sink = require('../db/file');
Expand Down
File renamed without changes.
2 changes: 1 addition & 1 deletion src/plugin.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
const logger = require('../src/logging/logger');
const { logger } = require('./logging/index');
const path = require('path');
const lpModule = import('load-plugin');
('use strict');
Expand Down
2 changes: 1 addition & 1 deletion src/proxy/chain.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
const proc = require('./processors');
const plugin = require('../plugin');
const logger = require('../../src/logging/logger');
const { logger } = require('../logging/index');

const pushActionChain = [
proc.push.parsePush,
Expand Down
2 changes: 1 addition & 1 deletion src/proxy/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ const bodyParser = require('body-parser');
const router = require('./routes').router;
const config = require('../config');
const db = require('../db');
const { logger } = require('../logging/logger');
const { logger } = require('../logging/index');
const { GIT_PROXY_SERVER_PORT: proxyHttpPort } = require('../config/env').Vars;

const options = {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
const Step = require('../../actions').Step;
const db = require('../../../db');
const logger = require('../../../../src/logging/logger');
const { logger } = require('../../../logging/index');

// Execute if the repo is approved
const exec = async (req, action, authorisedList = db.getRepos) => {
Expand Down
2 changes: 1 addition & 1 deletion src/proxy/routes/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ const proxy = require('express-http-proxy');
const router = express.Router();
const chain = require('../chain');

const logger = require('../../../src/logging/logger');
const { logger } = require('../../logging/index');

/**
* For a given Git HTTP request destined for a GitHub repo,
Expand Down
2 changes: 1 addition & 1 deletion src/service/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,10 @@ const express = require('express');
const session = require('express-session');
const http = require('http');
const cors = require('cors');
const { logger } = require('../logging/logger');
const app = express();
const rateLimit = require('express-rate-limit');
const lusca = require('lusca');
const { logger } = require('../logging/index');

const limiter = rateLimit({
windowMs: 15 * 60 * 1000, // 15 minutes
Expand Down
2 changes: 1 addition & 1 deletion src/service/routes/auth.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ const db = require('../../db');
const passportType = passport.type;
const generator = require('generate-password');
const passwordHash = require('password-hash');
const logger = require('../../logging/logger');
const { logger } = require('../../logging/index');

router.post('/login', passport.authenticate(passportType), (req, res) => {
res.send({
Expand Down
2 changes: 1 addition & 1 deletion src/service/routes/users.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
const express = require('express');
const router = new express.Router();
const db = require('../../db');
const logger = require('../../logging/logger');
const { logger } = require('../../logging/index');

router.get('/', async (req, res) => {
if (req.user) {
Expand Down

0 comments on commit dccdc78

Please sign in to comment.