Merge branch 'main' into fix-dos-when-pushing-unkown-repo

Author: andypols

.github/workflows/unused-dependencies.yml

@@ -21,7 +21,7 @@ jobs:
           node-version: '22.x'
       - name: 'Run depcheck'
         run: |
-          npx depcheck --skip-missing --ignores="tsx,@babel/*,@commitlint/*,eslint,eslint-*,husky,mocha,ts-mocha,ts-node,concurrently,nyc,prettier,typescript,tsconfig-paths,vite-tsconfig-paths"
+          npx depcheck --skip-missing --ignores="tsx,@babel/*,@commitlint/*,eslint,eslint-*,husky,mocha,ts-mocha,ts-node,concurrently,nyc,prettier,typescript,tsconfig-paths,vite-tsconfig-paths,history,@types/domutils"
           echo $?
           if [[ $? == 1 ]]; then
             echo "Unused dependencies or devDependencies found"

index.html

@@ -79,5 +79,5 @@
           To create a production bundle, use `npm run build` or `yarn build`.
         -->
   </body>
-  <script type="module" src="/src/index.jsx"></script>
+  <script type="module" src="/src/index.tsx"></script>
 </html>

package-lock.json

@@ -1,10 +1,10 @@
 {
   "name": "@finos/git-proxy",
-  "version": "1.18.2",
+  "version": "1.19.1",
   "description": "Deploy custom push protections and policies on top of Git.",
   "scripts": {
     "cli": "node ./packages/git-proxy-cli/index.js",
-    "client": "vite --config vite.config.js",
+    "client": "vite --config vite.config.ts",
     "clientinstall": "npm install --prefix client",
     "server": "tsx index.ts",
     "start": "concurrently \"npm run server\" \"npm run client\"",
@@ -88,14 +88,17 @@
     "@babel/preset-react": "^7.22.5",
     "@commitlint/cli": "^19.0.0",
     "@commitlint/config-conventional": "^19.0.0",
+    "@types/domutils": "^1.7.8",
     "@types/express": "^5.0.1",
     "@types/express-http-proxy": "^1.6.6",
     "@types/lodash": "^4.17.15",
     "@types/mocha": "^10.0.10",
     "@types/node": "^22.13.5",
+    "@types/react-dom": "^17.0.11",
+    "@types/react-html-parser": "^2.0.7",
     "@types/yargs": "^17.0.33",
-    "@typescript-eslint/eslint-plugin": "^8.26.1",
-    "@typescript-eslint/parser": "^8.26.1",
+    "@typescript-eslint/eslint-plugin": "^8.29.0",
+    "@typescript-eslint/parser": "^8.29.0",
     "@vitejs/plugin-react": "^4.0.2",
     "chai": "^4.2.0",
     "chai-http": "^4.3.0",

package.json

@@ -1,10 +1,6 @@
-import React, { useEffect } from 'react';
-import { Snackbar, TableCell, TableRow } from '@material-ui/core';
-import GridContainer from '../../../components/Grid/GridContainer';
-import GridItem from '../../../components/Grid/GridItem';
-import { CodeReviewIcon, LawIcon, PeopleIcon } from '@primer/octicons-react';
+// src/constants/languageColors.ts
 
-const colors = {
+export const languageColors: Record<string, string> = {
   '1C Enterprise': '#814CCC',
   '2-Dimensional Array': '#38761D',
   '4D': '#004289',
@@ -563,125 +559,3 @@ const colors = {
   ZIL: '#dc75e5',
   Zimpl: '#d67711',
 };
-
-import axios from 'axios';
-import moment from 'moment';
-import CodeActionButton from '../../../components/CustomButtons/CodeActionButton';
-
-export default function Repositories(props) {
-  const [github, setGitHub] = React.useState({});
-
-  const [errorMessage, setErrorMessage] = React.useState('');
-  const [snackbarOpen, setSnackbarOpen] = React.useState(false);
-
-  useEffect(() => {
-    getGitHubRepository();
-  }, [props.data.project, props.data.name]);
-
-  const getGitHubRepository = async () => {
-    await axios
-      .get(`https://api.github.com/repos/${props.data.project}/${props.data.name}`)
-      .then((res) => {
-        setGitHub(res.data);
-      })
-      .catch((error) => {
-        setErrorMessage(`Error fetching GitHub repository ${props.data.project}/${props.data.name}: ${error}`);
-        setSnackbarOpen(true);
-      });
-  };
-
-  const { project: org, name, proxyURL } = props?.data || {};
-  const cloneURL = `${proxyURL}/${org}/${name}.git`;
-
-  return (
-    <TableRow>
-      <TableCell>
-        <div style={{ padding: '15px' }}>
-          <a href={`/dashboard/repo/${props.data.name}`}>
-            <span style={{ fontSize: '17px' }}>
-              {props.data.project}/{props.data.name}
-            </span>
-          </a>
-          {github.parent && (
-            <span
-              style={{
-                fontSize: '11.5px',
-                display: 'block',
-                opacity: 0.8,
-              }}
-            >
-              Forked from{' '}
-              <a
-                style={{
-                  fontWeight: 'normal',
-                  color: 'inherit',
-                }}
-                href={github.parent.html_url}
-              >
-                {github.parent.full_name}
-              </a>
-            </span>
-          )}
-          {github.description && <p style={{ maxWidth: '80%' }}>{github.description}</p>}
-          <GridContainer>
-            {github.language && (
-              <GridItem>
-                <span
-                  style={{
-                    height: '12px',
-                    width: '12px',
-                    backgroundColor: `${colors[github.language]}`,
-                    borderRadius: '50px',
-                    display: 'inline-block',
-                    marginRight: '5px',
-                  }}
-                ></span>
-                {github.language}
-              </GridItem>
-            )}
-            {github.license && (
-              <GridItem>
-                <LawIcon size='small' />{' '}
-                <span style={{ marginLeft: '5px' }}>{github.license.spdx_id}</span>
-              </GridItem>
-            )}
-            <GridItem>
-              <PeopleIcon size='small' />{' '}
-              <span style={{ marginLeft: '5px' }}>{props.data.users?.canPush?.length || 0}</span>
-            </GridItem>
-            <GridItem>
-              <CodeReviewIcon size='small' />{' '}
-              <span style={{ marginLeft: '5px' }}>
-                {props.data.users?.canAuthorise?.length || 0}
-              </span>
-            </GridItem>
-            {(github.created_at || github.updated_at || github.pushed_at) && (
-              <GridItem>
-                Last updated{' '}
-                {moment
-                  .max([
-                    moment(github.created_at),
-                    moment(github.updated_at),
-                    moment(github.pushed_at),
-                  ])
-                  .fromNow()}
-              </GridItem>
-            )}
-          </GridContainer>
-        </div>
-      </TableCell>
-      <TableCell align='right'>
-        <div style={{ padding: '15px' }}>
-          <CodeActionButton cloneURL={cloneURL} />
-        </div>
-      </TableCell>
-      <Snackbar
-        anchorOrigin={{ vertical: 'top', horizontal: 'center' }}
-        open={snackbarOpen}
-        autoHideDuration={6000}
-        onClose={() => setSnackbarOpen(false)}
-        message={errorMessage}
-      />
-    </TableRow>
-  );
-}

src/ui/views/RepoList/Components/RepoOverview.jsx renamed to src/constants/languageColors.ts

@@ -0,0 +1,8 @@
+import { createContext } from 'react';
+import { UserContextType } from './ui/views/RepoDetails/RepoDetails';
+
+export const UserContext = createContext<UserContextType>({
+  user: {
+    admin: false,
+  },
+});

src/context.js

@@ -2,7 +2,7 @@ import fs from 'fs';
 import _ from 'lodash';
 import Datastore from '@seald-io/nedb';
 import { Action } from '../../proxy/actions/Action';
-import { toClass } from '../helper';
+import { toClass, trimTrailingDotGit } from '../helper';
 import * as repo from './repo';
 import { PushQuery } from '../types';
 
@@ -138,7 +138,7 @@ export const canUserCancelPush = async (id: string, user: string) => {
       return;
     }
 
-    const repoName = pushDetail.repoName.replace('.git', '');
+    const repoName = trimTrailingDotGit(pushDetail.repoName);
     const isAllowed = await repo.isUserPushAllowed(repoName, user);
 
     if (isAllowed) {
@@ -156,7 +156,8 @@ export const canUserApproveRejectPush = async (id: string, user: string) => {
       resolve(false);
       return;
     }
-    const repoName = action.repoName.replace('.git', '');
+
+    const repoName = trimTrailingDotGit(action.repoName);
     const isAllowed = await repo.canUserApproveRejectPushRepo(repoName, user);
 
     resolve(isAllowed);

src/context.ts

@@ -3,3 +3,21 @@ export const toClass = function (obj: any, proto: any) {
   obj.__proto__ = proto;
   return obj;
 };
+
+export const trimTrailingDotGit = (str: string): string => {
+  const target = '.git';
+  if (str.endsWith(target)) {
+    // extract string from 0 to the end minus the length of target
+    return str.slice(0, -target.length);
+  }
+  return str;
+};
+
+export const trimPrefixRefsHeads = (str: string): string => {
+  const target = 'refs/heads/';
+  if (str.startsWith(target)) {
+    // extract string from the end of the target to the end of str
+    return str.slice(target.length);
+  }
+  return str;
+};

src/db/file/pushes.ts

@@ -1,6 +1,6 @@
 import { connect, findDocuments, findOneDocument } from './helper';
 import { Action } from '../../proxy/actions';
-import { toClass } from '../helper';
+import { toClass, trimTrailingDotGit } from '../helper';
 import * as repo from './repo';
 import { Push, PushQuery } from '../types';
 
@@ -108,7 +108,7 @@ export const canUserApproveRejectPush = async (id: string, user: string) => {
       return;
     }
 
-    const repoName = action.repoName.replace('.git', '');
+    const repoName = trimTrailingDotGit(action.repoName);
     const isAllowed = await repo.canUserApproveRejectPushRepo(repoName, user);
 
     resolve(isAllowed);
@@ -123,7 +123,7 @@ export const canUserCancelPush = async (id: string, user: string) => {
       return;
     }
 
-    const repoName = pushDetail.repoName.replace('.git', '');
+    const repoName = trimTrailingDotGit(pushDetail.repoName);
     const isAllowed = await repo.isUserPushAllowed(repoName, user);
 
     if (isAllowed) {