Skip to content

Commit 602548c

Browse files
authored
Merge pull request #1194 from hackmdio/upgrade-dependency
Upgrade dependency to fix vulnerabilities issue
2 parents 7a40fdc + 97aecfa commit 602548c

File tree

7 files changed

+854
-895
lines changed

7 files changed

+854
-895
lines changed

lib/models/note.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ var path = require('path')
55
var LZString = require('@hackmd/lz-string')
66
var base64url = require('base64url')
77
var md = require('markdown-it')()
8-
var metaMarked = require('meta-marked')
8+
var metaMarked = require('@hackmd/meta-marked')
99
var cheerio = require('cheerio')
1010
var shortId = require('shortid')
1111
var Sequelize = require('sequelize')

lib/web/auth/mattermost/index.js

Lines changed: 10 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,17 @@
11
'use strict'
2-
2+
require('babel-polyfill')
3+
require('isomorphic-fetch');
34
const Router = require('express').Router
45
const passport = require('passport')
5-
const Mattermost = require('mattermost')
6+
const MattermostClient = require('mattermost-redux/client/client4').default
67
const OAuthStrategy = require('passport-oauth2').Strategy
78
const config = require('../../../config')
89
const { setReturnToFromReferer, passportGeneralCallback } = require('../utils')
910

10-
const mattermost = new Mattermost.Client()
11-
1211
let mattermostAuth = module.exports = Router()
1312

13+
const mattermostClient = new MattermostClient()
14+
1415
let mattermostStrategy = new OAuthStrategy({
1516
authorizationURL: config.mattermost.baseURL + '/oauth/authorize',
1617
tokenURL: config.mattermost.baseURL + '/oauth/access_token',
@@ -20,17 +21,11 @@ let mattermostStrategy = new OAuthStrategy({
2021
}, passportGeneralCallback)
2122

2223
mattermostStrategy.userProfile = (accessToken, done) => {
23-
mattermost.setUrl(config.mattermost.baseURL)
24-
mattermost.token = accessToken
25-
mattermost.useHeaderToken()
26-
mattermost.getMe(
27-
(data) => {
28-
done(null, data)
29-
},
30-
(err) => {
31-
done(err)
32-
}
33-
)
24+
mattermostClient.setUrl(config.mattermost.baseURL)
25+
mattermostClient.setToken(accessToken)
26+
mattermostClient.getMe()
27+
.then((data) => done(null, data))
28+
.catch((err) => done(err))
3429
}
3530

3631
passport.use(mattermostStrategy)

0 commit comments

Comments
 (0)