Feat/email login flow (#480)

* build(deps): bump file-type from 16.5.3 to 16.5.4 (#475)

Bumps [file-type](https://github.com/sindresorhus/file-type) from 16.5.3 to 16.5.4.
- [Release notes](https://github.com/sindresorhus/file-type/releases)
- [Commits](sindresorhus/file-type@v16.5.3...v16.5.4)

---
updated-dependencies:
- dependency-name: file-type
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* fix: package.json & package-lock.json to reduce vulnerabilities (#476)

The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JS-SEQUELIZE-2959225

* build(deps): bump vm2 from 3.9.5 to 3.9.7 (#350)

Bumps [vm2](https://github.com/patriksimek/vm2) from 3.9.5 to 3.9.7.
- [Release notes](https://github.com/patriksimek/vm2/releases)
- [Changelog](https://github.com/patriksimek/vm2/blob/master/CHANGELOG.md)
- [Commits](patriksimek/vm2@3.9.5...3.9.7)

---
updated-dependencies:
- dependency-name: vm2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Chore: remove site links from description (#482)

* Fix: update resource room (#481)

* 0.10.0

* fix: remove unnecessary update step (#487)

* 0.10.1

* Chore: update commit message to include user id

* Feat: add login and verify endpoints

* Fix: model relations and alias

* Feat: add findSitesByUserId

* Feat: add site retrieval for email and admin users

* Fix: hasAccessToSite

* Fix: update email/mobile by isomer id

* Chore: update error message

* Fix: await check for whitelist

* Chore: add mockSessionData for email login

* Fix: SiteService behaviour for email users with no whitelisted sites

* Test: update sitesservice tests

* Test: add new authservice tests and fix existing tests

* Fix: update user model to allow null in github field

* Fix: update test fixture

* Fix: update user test suite

* Chore: remove unused endpoint

* Fix: rebase errors

* Chore: remove unnecessary message in test

* Chore: remove unnecessary userId field

* Nit: rename variable

* Refactor: shift site retrieval for email users into helper method

* Chore: spacing and remove unused var

* Fix: tests

* Tests: add new authorizationMiddlewareService test

* fix: remove resources_name and add support for url (#490)

* fix: remove resources_name and add support for url

* fix: display url parameter as domain but store with https scheme

* fix: resolve failing tests

* Chore: flip conditional

* Refactor: shift order of getSites to make it easier to understand

* Test: add new auth router tests

* Feat: add integration tests for getSites

* Fix: failing requests for getLastUpdated and getStagingUrl

* Nit: add comment

* Nit: test name and var name

* chore(mocks/axios): remove extra stuff

* test(sites.spec): refactor specs for clarity

* Fix: update settings

* Nit: update comment

* Fix: tests

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Snyk bot <[email protected]>
Co-authored-by: Hsu Zhong Jun <[email protected]>
Co-authored-by: seaerchin <[email protected]>

CHANGELOG.md

@@ -4,8 +4,26 @@ All notable changes to this project will be documented in this file. Dates are d
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+#### [v0.10.1](https://github.com/isomerpages/isomercms-backend/compare/v0.10.0...v0.10.1)
+
+- fix: remove unnecessary update step [`#487`](https://github.com/isomerpages/isomercms-backend/pull/487)
+- 0.10.0 [`#484`](https://github.com/isomerpages/isomercms-backend/pull/484)
+
+#### [v0.10.0](https://github.com/isomerpages/isomercms-backend/compare/v0.9.0...v0.10.0)
+
+> 11 August 2022
+
+- Fix: update resource room [`#481`](https://github.com/isomerpages/isomercms-backend/pull/481)
+- Chore: remove site links from description [`#482`](https://github.com/isomerpages/isomercms-backend/pull/482)
+- build(deps): bump vm2 from 3.9.5 to 3.9.7 [`#350`](https://github.com/isomerpages/isomercms-backend/pull/350)
+- fix: package.json & package-lock.json to reduce vulnerabilities [`#476`](https://github.com/isomerpages/isomercms-backend/pull/476)
+- build(deps): bump file-type from 16.5.3 to 16.5.4 [`#475`](https://github.com/isomerpages/isomercms-backend/pull/475)
+- 0.9.0 [`#473`](https://github.com/isomerpages/isomercms-backend/pull/473)
+
 #### [v0.9.0](https://github.com/isomerpages/isomercms-backend/compare/v0.8.0...v0.9.0)
 
+> 14 July 2022
+
 - Misc Backend Cleanup 1 [`#470`](https://github.com/isomerpages/isomercms-backend/pull/470)
 - 0.8.0 [`#468`](https://github.com/isomerpages/isomercms-backend/pull/468)
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "isomercms",
-  "version": "0.9.0",
+  "version": "0.10.1",
   "private": true,
   "scripts": {
     "build": "tsc -p tsconfig.build.json",
@@ -41,7 +41,7 @@
     "dotenv": "^16.0.1",
     "exponential-backoff": "^3.1.0",
     "express": "~4.16.1",
-    "file-type": "^16.5.3",
+    "file-type": "^16.5.4",
     "helmet": "^4.6.0",
     "http-errors": "~1.8.0",
     "is-svg": "^4.3.1",
@@ -60,7 +60,7 @@
     "pg-connection-string": "^2.5.0",
     "query-string": "^6.14.1",
     "reflect-metadata": "^0.1.13",
-    "sequelize": "^6.17.0",
+    "sequelize": "^6.21.2",
     "sequelize-typescript": "^2.1.3",
     "serialize-error": "^7.0.1",
     "slugify": "^1.6.0",

src/__mocks__/axios.ts

@@ -1,6 +1,3 @@
 import mockAxios from "jest-mock-axios"
 
-mockAxios.interceptors.request.use(jest.fn())
-mockAxios.interceptors.response.use(jest.fn())
-
 export default mockAxios

src/database/migrations/20220803091224-change-users-github-allow-null.js

@@ -0,0 +1,23 @@
+module.exports = {
+  async up(queryInterface, Sequelize) {
+    await queryInterface.changeColumn("users", "github_id", {
+      allowNull: true,
+      unique: true,
+      type: Sequelize.TEXT,
+      validate: {
+        notEmpty: true,
+      },
+    })
+  },
+
+  async down(queryInterface, Sequelize) {
+    await queryInterface.changeColumn("users", "github_id", {
+      allowNull: false,
+      unique: true,
+      type: Sequelize.TEXT,
+      validate: {
+        notEmpty: true,
+      },
+    })
+  },
+}

src/database/models/Site.ts

@@ -68,6 +68,7 @@ export class Site extends Model {
     onUpdate: "CASCADE",
     onDelete: "CASCADE",
     through: () => SiteMember,
+    as: "site_members",
   })
   users!: User[]
 
@@ -80,6 +81,8 @@ export class Site extends Model {
   @ForeignKey(() => User)
   creatorId!: number
 
-  @BelongsTo(() => User)
+  @BelongsTo(() => User, {
+    as: "site_creator",
+  })
   creator!: User
 }

src/database/models/User.ts

@@ -31,7 +31,7 @@ export class User extends Model {
   email?: string | null
 
   @Column({
-    allowNull: false,
+    allowNull: true,
     unique: true,
     type: DataType.TEXT,
     validate: {
@@ -61,13 +61,16 @@ export class User extends Model {
   @DeletedAt
   deletedAt?: Date
 
-  @BelongsToMany(() => User, {
+  @BelongsToMany(() => Site, {
     onUpdate: "CASCADE",
     onDelete: "CASCADE",
     through: () => SiteMember,
+    as: "site_members",
   })
   sites!: Site[]
 
-  @HasMany(() => Site)
+  @HasMany(() => Site, {
+    as: "sites_created",
+  })
   sitesCreated?: Site[]
 }

src/fixtures/config.js

@@ -118,7 +118,7 @@ const configContent = {
 const configSha = "configsha"
 
 const configResponse = {
-  url: configContent.url,
+  url: configContent.url.replace("https://", ""),
   title: configContent.title,
   description: configContent.description,
   favicon: configContent.favicon,
@@ -127,7 +127,6 @@ const configResponse = {
   facebook_pixel: configContent["facebook-pixel"],
   google_analytics: configContent.google_analytics,
   linkedin_insights: configContent["linkedin-insights"],
-  resources_name: configContent.resources_name,
   colors: configContent.colors,
 }
 

src/fixtures/sessionData.ts

@@ -4,7 +4,7 @@ import UserWithSiteSessionData from "@root/classes/UserWithSiteSessionData"
 
 export const mockAccessToken = "mockAccessToken"
 export const mockGithubId = "mockGithubId"
-export const mockIsomerUserId = "mockIsomerUserId"
+export const mockIsomerUserId = "1"
 export const mockEmail = "mockEmail"
 export const mockTreeSha = "mockTreeSha"
 export const mockCurrentCommitSha = "mockCurrentCommitSha"
@@ -34,3 +34,12 @@ export const mockGithubSessionData = new GithubSessionData({
   treeSha: mockTreeSha,
   currentCommitSha: mockCurrentCommitSha,
 })
+export const mockSessionDataEmailUser = new UserSessionData({
+  isomerUserId: mockIsomerUserId,
+  email: mockEmail,
+})
+export const mockSessionDataEmailUserWithSite = new UserWithSiteSessionData({
+  isomerUserId: mockIsomerUserId,
+  email: mockEmail,
+  siteName: mockSiteName,
+})