Production release v1.1.0 (#126)

* Add pikaratikka to the maps

* Add icons and colors for pikaratikka

* Fix typo on color code

* Change pikaratikka to updated color

* MM-523: Fix rendering error about minimum radius on circle vectors (#109)

* #25217: Initialize this.root in component constructors

* Cleanup

* 25664: Prevent large generations timeout and left on pending state

* Bump hsl-map-style version to 1.1.3 (#117)

* AB#48634: Bump hsl-map-style version to 1.2.0 (#118)

* zoneSymbol positioning fix (#115)

* AB#57971 improve access control (#120)

* AB#57971: Switch user domain validation to validation library

* Add dependency changes

* AB#56581: Add version numbering and check against prod (#122)

* AB#56568: Change license to AGPL-3.0-only (#124)

* react updates (#116)

* AB#57970: Switch prod login provider (#125)

* v1.1.0

---------

Co-authored-by: Juho Hänninen <juho.hanninen@cgi.com>
Co-authored-by: Anton Jyrkiäinen <anton.jyrkiainen@helsinki.fi>

Author: 3 people

.env.prod

@@ -12,11 +12,11 @@ AZURE_STORAGE_KEY=
 AZURE_FONTS_SAS_URL=
 AZURE_UPLOAD_CONTAINER=routemap-prod
 
-CLIENT_ID=7833861618225795
+CLIENT_ID=0704589208046070
 CLIENT_SECRET=
 API_CLIENT_ID=2411181397763460
 API_CLIENT_SECRET=
 REDIRECT_URI=https://kartat.hsl.fi/kartta
-LOGIN_PROVIDER_URI=https://hslid-uat.cinfra.fi
+LOGIN_PROVIDER_URI=https://id.hsl.fi
 DOMAINS_ALLOWED_TO_GENERATE=
 DOMAINS_ALLOWED_TO_LOGIN=

.env.stage

@@ -0,0 +1,22 @@
+name: Check version numbering is updated prior to merging
+
+on:
+  pull_request:
+    branches:
+      - master
+
+jobs:
+  check_version:
+    runs-on: ubuntu-latest
+    steps:
+    - name: Checkout code
+      uses: actions/checkout@v4
+    - name: Check if version has been updated
+      id: check
+      uses: EndBug/version-check@v2
+    - name: Fail if version is not changed prior to merge
+      if: steps.check.outputs.changed == 'false'
+      uses: actions/github-script@v7
+      with:
+        script: |
+          core.setFailed('No version number change found. Run `yarn version`-command to upgrade version before merge can be completed.')

.github/workflows/check_version.yml

@@ -1,6 +1,6 @@
 {
   "name": "hsl-routemap-server",
-  "version": "0.1.0",
+  "version": "1.1.0",
   "description": "HSL Routemap server",
   "main": "index.js",
   "scripts": {
@@ -30,7 +30,7 @@
     "url": "git+https://github.com/HSLdevcom/hsl-routemap-server.git"
   },
   "author": "",
-  "license": "MIT",
+  "license": "AGPL-3.0-only",
   "bugs": {
     "url": "https://github.com/HSLdevcom/hsl-routemap-server/issues"
   },
@@ -96,13 +96,14 @@
     "pg": "^8.7.3",
     "prop-types": "^15.6.0",
     "puppeteer": "^15.4.1",
-    "react": "16.8.6",
+    "react": "18.2.0",
     "react-apollo": "^2.0.1",
-    "react-dom": "16.8.6",
+    "react-dom": "18.2.0",
     "recompose": "^0.30.0",
     "segseg": "^0.2.2",
     "serve": "^13.0.2",
     "uuid": "^3.1.0",
+    "validator": "^13.15.0",
     "viewport-mercator-project": "^4.1.1"
   }
 }

LICENSE

@@ -1,19 +1,26 @@
 const { get, last, clone } = require('lodash');
 const AuthService = require('./authService');
+const validator = require('validator');
 
 const { DOMAINS_ALLOWED_TO_LOGIN, ROUTEMAP_TEST_GROUP } = require('../../constants');
 
 const allowedDomains = DOMAINS_ALLOWED_TO_LOGIN.split(',');
 
 const hasAllowedDomain = async (userInfo) => {
-  const groupNames = get(userInfo, 'groups');
-  const domain = last(userInfo.email.toLowerCase().split('@')) || '';
+  const groups = get(userInfo, 'groups');
 
-  if (groupNames.includes(ROUTEMAP_TEST_GROUP)) {
+  const emailValidationOptions = {
+    host_whitelist: allowedDomains,
+  };
+
+  if (groups.includes(ROUTEMAP_TEST_GROUP)) {
     return true;
   }
 
-  if (!allowedDomains.includes(domain)) {
+  if (
+    !validator.isEmail(userInfo.email, emailValidationOptions) &&
+    !groups.includes(ROUTEMAP_TEST_GROUP)
+  ) {
     console.log(`User does not have allowed domain. Logging out.`);
     return false;
   }

package.json

@@ -1,59 +1,50 @@
-import React, { Component } from 'react';
+import React, { useState, useEffect } from 'react';
 import renderQueue from 'util/renderQueue';
 
-const hocFactory = propName => WrappedComponent =>
-  class PromiseWrapper extends Component {
-    constructor(props) {
-      super(props);
-      this.state = { loading: !!props[propName] };
-    }
-
-    componentDidMount() {
-      if (this.props[propName]) {
-        this.handlePromise(this.props[propName]);
-      }
-    }
+const hocFactory = (propName) => (WrappedComponent) => (props) => {
+  const [state, setState] = useState({
+    loading: !!props[propName],
+    value: null,
+    error: null,
+  });
 
-    componentWillReceiveProps(nextProps) {
-      if (nextProps[propName] && nextProps[propName] !== this.props[propName]) {
-        this.setState({ loading: true });
-        this.handlePromise(nextProps[propName]);
-      }
-    }
-
-    componentWillUnmount() {
-      this.promise = null;
-    }
+  useEffect(() => {
+    let isMounted = true;
+    const promise = props[propName];
 
-    handlePromise(promise) {
-      this.promise = promise;
+    if (promise) {
+      setState((prevState) => ({ ...prevState, loading: true }));
       renderQueue.add(promise);
+
       promise
-        .then(value => {
-          const callback = () => renderQueue.remove(promise);
-          if (this.promise !== promise) {
-            callback();
-          } else {
-            this.setState({ value, loading: false }, callback);
+        .then((value) => {
+          if (isMounted) {
+            setState({ value, loading: false, error: null });
+            renderQueue.remove(promise);
           }
         })
-        .catch(error => {
-          const callback = () => renderQueue.remove(promise, { error });
-          if (this.promise !== promise) {
-            callback();
-          } else {
-            this.setState({ error, loading: false }, callback);
+        .catch((error) => {
+          if (isMounted) {
+            setState({ error, loading: false });
+            renderQueue.remove(promise, { error });
           }
         });
     }
 
-    render() {
-      if (this.state.loading || this.state.error) {
-        return null;
-      }
-      const props = { ...this.props, [propName]: this.state.value };
-      return <WrappedComponent {...props} />;
-    }
-  };
+    return () => {
+      isMounted = false;
+      renderQueue.remove(promise);
+    };
+  }, [props[propName]]);
+
+  const { loading, error, value } = state;
+
+  if (loading || error) {
+    return null;
+  }
+
+  const newProps = { ...props, [propName]: value };
+  return <WrappedComponent {...newProps} />;
+};
 
 export default hocFactory;

scripts/auth/authEndpoints.js

@@ -7475,7 +7475,7 @@ prompt@0.2.14:
     utile "0.2.x"
     winston "0.8.x"
 
-prop-types@^15.6.0, prop-types@^15.6.1, prop-types@^15.6.2, prop-types@^15.7.2, prop-types@^15.8.1:
+prop-types@^15.6.0, prop-types@^15.6.1, prop-types@^15.7.2, prop-types@^15.8.1:
   version "15.8.1"
   resolved "https://registry.yarnpkg.com/prop-types/-/prop-types-15.8.1.tgz#67d87bf1a694f48435cf332c24af10214a3140b5"
   integrity sha512-oj87CgZICdulUohogVAR7AjlC0327U4el4L6eAvOqCeudMDVU0NThNaV+b9Df4dXgSP1gXMTnPdhfe/2qDH5cg==
@@ -7687,15 +7687,13 @@ react-apollo@^2.0.1:
     ts-invariant "^0.4.2"
     tslib "^1.9.3"
 
-react-dom@16.8.6:
-  version "16.8.6"
-  resolved "https://registry.yarnpkg.com/react-dom/-/react-dom-16.8.6.tgz#71d6303f631e8b0097f56165ef608f051ff6e10f"
-  integrity sha512-1nL7PIq9LTL3fthPqwkvr2zY7phIPjYrT0jp4HjyEQrEROnw4dG41VVwi/wfoCneoleqrNX7iAD+pXebJZwrwA==
+react-dom@18.2.0:
+  version "18.2.0"
+  resolved "https://registry.yarnpkg.com/react-dom/-/react-dom-18.2.0.tgz#22aaf38708db2674ed9ada224ca4aa708d821e3d"
+  integrity sha512-6IMTriUmvsjHUjNtEDudZfuDQUoWXVxKHhlEGSk81n4YFS+r/Kl99wXiwlVXtPBtJenozv2P+hxDsw9eA7Xo6g==
   dependencies:
     loose-envify "^1.1.0"
-    object-assign "^4.1.1"
-    prop-types "^15.6.2"
-    scheduler "^0.13.6"
+    scheduler "^0.23.0"
 
 react-hot-loader@^4.13.1:
   version "4.13.1"
@@ -7721,15 +7719,12 @@ react-lifecycles-compat@^3.0.2, react-lifecycles-compat@^3.0.4:
   resolved "https://registry.yarnpkg.com/react-lifecycles-compat/-/react-lifecycles-compat-3.0.4.tgz#4f1a273afdfc8f3488a8c516bfda78f872352362"
   integrity sha512-fBASbA6LnOU9dOU2eW7aQ8xmYBSXUIWr+UmF9b1efZBazGNO+rcXT/icdKnYm2pTwcRylVUYwW7H1PHfLekVzA==
 
-react@16.8.6:
-  version "16.8.6"
-  resolved "https://registry.yarnpkg.com/react/-/react-16.8.6.tgz#ad6c3a9614fd3a4e9ef51117f54d888da01f2bbe"
-  integrity sha512-pC0uMkhLaHm11ZSJULfOBqV4tIZkx87ZLvbbQYunNixAAvjnC+snJCg0XQXn9VIsttVsbZP/H/ewzgsd5fxKXw==
+react@18.2.0:
+  version "18.2.0"
+  resolved "https://registry.yarnpkg.com/react/-/react-18.2.0.tgz#555bd98592883255fa00de14f1151a917b5d77d5"
+  integrity sha512-/3IjMdb2L9QbBdWiW5e3P2/npwMBaU9mHCSCUzNln0ZCYbcfTsGbTJrU/kGemdH2IWmB2ioZ+zkxtmq6g09fGQ==
   dependencies:
     loose-envify "^1.1.0"
-    object-assign "^4.1.1"
-    prop-types "^15.6.2"
-    scheduler "^0.13.6"
 
 read@1.0.x:
   version "1.0.7"
@@ -8149,13 +8144,12 @@ sax@>=0.6.0:
   resolved "https://registry.yarnpkg.com/sax/-/sax-1.2.4.tgz#2816234e2378bddc4e5354fab5caa895df7100d9"
   integrity sha512-NqVDv9TpANUjFm0N8uM5GxL36UgKi9/atZw+x7YFnQ8ckwFGKrl4xX4yWtrey3UJm5nP1kUbnYgLopqWNSRhWw==
 
-scheduler@^0.13.6:
-  version "0.13.6"
-  resolved "https://registry.yarnpkg.com/scheduler/-/scheduler-0.13.6.tgz#466a4ec332467b31a91b9bf74e5347072e4cd889"
-  integrity sha512-IWnObHt413ucAYKsD9J1QShUKkbKLQQHdxRyw73sw4FN26iWr3DY/H34xGPe4nmL1DwXyWmSWmMrA9TfQbE/XQ==
+scheduler@^0.23.0:
+  version "0.23.0"
+  resolved "https://registry.yarnpkg.com/scheduler/-/scheduler-0.23.0.tgz#ba8041afc3d30eb206a487b6b384002e4e61fdfe"
+  integrity sha512-CtuThmgHNg7zIZWAXi3AsyIzA3n4xx7aNyjwC2VJldO2LMVDhFK+63xGqq6CsJH4rTAt6/M+N4GhZiDYPx9eUw==
   dependencies:
     loose-envify "^1.1.0"
-    object-assign "^4.1.1"
 
 schema-utils@^1.0.0:
   version "1.0.0"
@@ -9408,6 +9402,11 @@ uuid@^9.0.0:
   resolved "https://registry.yarnpkg.com/uuid/-/uuid-9.0.0.tgz#592f550650024a38ceb0c562f2f6aa435761efb5"
   integrity sha512-MXcSTerfPa4uqyzStbRoTgt5XIe3x5+42+q1sDuy3R5MDk66URdLMOZe5aPX/SQd+kuYAh0FdP/pO28IkQyTeg==
 
+validator@^13.15.0:
+  version "13.15.0"
+  resolved "https://registry.yarnpkg.com/validator/-/validator-13.15.0.tgz#2dc7ce057e7513a55585109eec29b2c8e8c1aefd"
+  integrity sha512-36B2ryl4+oL5QxZ3AzD0t5SsMNGvTtQHpjgFO5tbNxfXbMFkY822ktCDe1MnlqV3301QQI9SLHDNJokDI+Z9pA==
+
 vary@^1.1.2, vary@~1.1.2:
   version "1.1.2"
   resolved "https://registry.yarnpkg.com/vary/-/vary-1.1.2.tgz#2299f02c6ded30d4a5961b0b9f74524a18f634fc"