Skip to content

Commit 8a405e9

Browse files
Christian SchmidtBrutus5000
authored andcommitted
Drop account creation and password reset
1 parent 785a41e commit 8a405e9

File tree

12 files changed

+16
-345
lines changed

12 files changed

+16
-345
lines changed

.env.example

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
PORT=3000
22
HOST=http://localhost:3000
3-
API_URL=https://api.test.faforever.com
4-
OAUTH_URL=https://hydra.test.faforever.com
5-
WP_URL=https://direct.faforever.com
3+
BASE_DOMAIN=faforever.com
64
CALLBACK=auth
75
OAUTH_M2M_CLIENT_ID=faf-website-public
86
OAUTH_M2M_CLIENT_SECRET=banana

src/backend/AppKernel.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,9 @@ class AppKernel {
4545

4646
bootstrapExpress() {
4747
this.expressApp = express()
48-
this.expressApp.locals.discordUrl = 'https://discord.gg/mXahVSKGVb'
49-
this.expressApp.locals.forumUrl = 'https://forum.faforever.com/'
48+
this.expressApp.locals.discordUrl = appConfig.discordUrl
49+
this.expressApp.locals.forumUrl = appConfig.forumUrl
50+
this.expressApp.locals.userServiceUrl = appConfig.userServiceUrl
5051
this.expressApp.locals.clanInvitations = {}
5152
this.expressApp.use((req, res, next) => {
5253
res.locals.navLinks = []

src/backend/config/app.js

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
require('dotenv').config()
22

3-
const oauthUrl = process.env.OAUTH_URL || 'https://hydra.faforever.com'
3+
const oauthUrl =
4+
process.env.OAUTH_URL || `https://hydra.${process.env.BASE_DOMAIN}`
45

56
const appConfig = {
67
nodeEnv: process.env.NODE_ENV || 'production',
@@ -23,8 +24,15 @@ const appConfig = {
2324
clientSecret: process.env.OAUTH_M2M_CLIENT_SECRET || 'banana',
2425
url: oauthUrl,
2526
},
26-
apiUrl: process.env.API_URL || 'https://api.faforever.com',
27-
wordpressUrl: process.env.WP_URL || 'https://direct.faforever.com',
27+
userServiceUrl:
28+
process.env.USER_SERVICE_URL ||
29+
`https://user.${process.env.BASE_DOMAIN}`,
30+
apiUrl: process.env.API_URL || `https://api.${process.env.BASE_DOMAIN}`,
31+
forumUrl:
32+
process.env.FORUM_URL || `https://forum.${process.env.BASE_DOMAIN}`,
33+
wordpressUrl:
34+
process.env.WP_URL || `https://direct.${process.env.BASE_DOMAIN}`,
35+
discordUrl: process.env.DISCORD_URL || 'https://discord.gg/mXahVSKGVb',
2836
extractorInterval: process.env.EXTRACTOR_INTERVAL || 5,
2937
playerCountInterval: process.env.PLAYER_COUNT_INTERVAL || 15,
3038
recaptchaKey: process.env.RECAPTCHA_SITE_KEY || 'test',

src/backend/routes/views/account/get/confirmPasswordReset.js

Lines changed: 0 additions & 61 deletions
This file was deleted.

src/backend/routes/views/account/get/createAccount.js

Lines changed: 0 additions & 25 deletions
This file was deleted.

src/backend/routes/views/accountRouter.js

Lines changed: 0 additions & 45 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,6 @@
11
const express = require('../../ExpressApp')
22
const router = express.Router()
3-
4-
const { query } = require('express-validator')
53
const middlewares = require('../middleware')
6-
const url = require('url')
74

85
router.get(
96
'/linkGog',
@@ -60,54 +57,12 @@ router.post(
6057
require('./account/post/changeUsername')
6158
)
6259

63-
router.get(
64-
'/password/confirmReset',
65-
[
66-
query('token').notEmpty().withMessage('Missing token'),
67-
query('username').notEmpty().withMessage('Missing username'),
68-
],
69-
require('./account/get/confirmPasswordReset')
70-
)
71-
router.post(
72-
'/password/confirmReset',
73-
require('./account/post/confirmPasswordReset')
74-
)
75-
76-
router.get(
77-
'/requestPasswordReset',
78-
require('./account/get/requestPasswordReset')
79-
)
80-
router.post(
81-
'/requestPasswordReset',
82-
require('./account/post/requestPasswordReset')
83-
)
84-
85-
// still used in other applications (user-service, game-client etc.)
86-
router.get('/password/reset', (req, res) =>
87-
res.redirect('/account/requestPasswordReset')
88-
)
89-
router.get('/confirmPasswordReset', (req, res) => {
90-
res.redirect(
91-
url.format({
92-
pathname: '/account/password/confirmReset',
93-
query: req.query,
94-
})
95-
)
96-
})
97-
98-
router.get('/register', require('./account/get/register'))
99-
router.post('/register', require('./account/post/register'))
100-
101-
router.get('/activate', require('./account/get/activate'))
102-
router.post('/activate', require('./account/post/activate'))
103-
10460
router.get('/checkUsername', require('./checkUsername'))
10561
router.get(
10662
'/resync',
10763
middlewares.isAuthenticated(),
10864
require('./account/get/resync')
10965
)
110-
router.get('/create', require('./account/get/createAccount'))
11166
router.get(
11267
'/link',
11368
middlewares.isAuthenticated(),

src/backend/templates/layouts/default.pug

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -149,7 +149,7 @@ html(lang='en')
149149
li.navList LOGIN
150150
.navContainer.navEnd
151151
ul.navItem
152-
a(href='/account/register')
152+
a(href=`${userServiceUrl}/register`)
153153
li.navList REGISTER
154154

155155
if appGlobals.loggedInUser

src/backend/templates/views/account/confirmPasswordReset.pug

Lines changed: 0 additions & 26 deletions
This file was deleted.

src/backend/templates/views/account/createAccount.pug

Lines changed: 0 additions & 17 deletions
This file was deleted.

src/backend/templates/views/account/register.pug

Lines changed: 0 additions & 50 deletions
This file was deleted.

0 commit comments

Comments
 (0)