Proper clientId/secret/token caching, MastoAPI registration

This commit is contained in:
Henry Jameson 2019-05-22 19:13:41 +03:00
parent 233506f6c1
commit e7a2a7267d
10 changed files with 138 additions and 92 deletions

View file

@ -31,8 +31,7 @@ module.exports = {
'vue/require-prop-types': 1, 'vue/require-prop-types': 1,
'vue/no-use-v-if-with-v-for': 1, 'vue/no-use-v-if-with-v-for': 1,
'indent': 1, 'indent': 1,
'import/first': 1, // ???? 'import/first': 1,
'import-first': 1,
'object-curly-spacing': 1, 'object-curly-spacing': 1,
'prefer-promise-reject-errors': 1, 'prefer-promise-reject-errors': 1,
'eol-last': 1, 'eol-last': 1,

View file

@ -3,6 +3,8 @@ import VueRouter from 'vue-router'
import routes from './routes' import routes from './routes'
import App from '../App.vue' import App from '../App.vue'
import { windowWidth } from '../services/window_utils/window_utils' import { windowWidth } from '../services/window_utils/window_utils'
import { getOrCreateApp, getClientToken } from '../services/new_api/oauth.js'
import backendInteractorService from '../services/backend_interactor_service/backend_interactor_service.js'
const getStatusnetConfig = async ({ store }) => { const getStatusnetConfig = async ({ store }) => {
try { try {
@ -188,6 +190,17 @@ const getCustomEmoji = async ({ store }) => {
} }
} }
const getAppSecret = async ({ store }) => {
const { state, commit } = store
const { oauth, instance } = state
return getOrCreateApp({ ...oauth, instance: instance.server, commit })
.then((app) => getClientToken({ ...app, instance: instance.server }))
.then((token) => {
commit('setClientToken', token.access_token)
commit('setBackendInteractor', backendInteractorService(store.getters.getToken()))
})
}
const getNodeInfo = async ({ store }) => { const getNodeInfo = async ({ store }) => {
try { try {
const res = await window.fetch('/nodeinfo/2.0.json') const res = await window.fetch('/nodeinfo/2.0.json')
@ -228,7 +241,7 @@ const setConfig = async ({ store }) => {
const apiConfig = configInfos[0] const apiConfig = configInfos[0]
const staticConfig = configInfos[1] const staticConfig = configInfos[1]
await setSettings({ store, apiConfig, staticConfig }) await setSettings({ store, apiConfig, staticConfig }).then(getAppSecret({ store }))
} }
const checkOAuthToken = async ({ store }) => { const checkOAuthToken = async ({ store }) => {

View file

@ -11,22 +11,26 @@ const LoginForm = {
}, },
methods: { methods: {
oAuthLogin () { oAuthLogin () {
oauthApi.login({ const data = {
oauth: this.$store.state.oauth, ...this.$store.state.oauth,
instance: this.$store.state.instance.server, instance: this.$store.state.instance.server,
commit: this.$store.commit commit: this.$store.commit
}) }
oauthApi.getOrCreateApp(data)
.then((app) => { oauthApi.login({ ...app, ...data }) })
}, },
submit () { submit () {
const data = { const data = {
oauth: this.$store.state.oauth, ...this.$store.state.oauth,
instance: this.$store.state.instance.server instance: this.$store.state.instance.server,
commit: this.$store.commit
} }
this.clearError() this.clearError()
oauthApi.getOrCreateApp(data).then((app) => { oauthApi.getOrCreateApp(data).then((app) => {
oauthApi.getTokenWithCredentials( oauthApi.getTokenWithCredentials(
{ {
app, ...app,
instance: data.instance, instance: data.instance,
username: this.user.username, username: this.user.username,
password: this.user.password password: this.user.password

View file

@ -5,7 +5,7 @@ const oac = {
mounted () { mounted () {
if (this.code) { if (this.code) {
oauth.getToken({ oauth.getToken({
app: this.$store.state.oauth, ...this.$store.state.oauth,
instance: this.$store.state.instance.server, instance: this.$store.state.instance.server,
code: this.code code: this.code
}).then((result) => { }).then((result) => {

View file

@ -1,17 +1,26 @@
const oauth = { const oauth = {
state: { state: {
client_id: false, clientId: false,
client_secret: false, clientSecret: false,
token: false token: false,
clientToken: false
}, },
mutations: { mutations: {
setClientData (state, data) { setClientData (state, { clientId, clientSecret }) {
state.client_id = data.client_id state.clientId = clientId
state.client_secret = data.client_secret state.clientSecret = clientSecret
},
setClientToken (state, token) {
state.clientToken = token
}, },
setToken (state, token) { setToken (state, token) {
state.token = token state.token = token
} }
},
getters: {
getToken: state => () => {
return state.token || state.clientToken
}
} }
} }

View file

@ -3,7 +3,6 @@ import userSearchApi from '../services/new_api/user_search.js'
import { compact, map, each, merge, last, concat, uniq } from 'lodash' import { compact, map, each, merge, last, concat, uniq } from 'lodash'
import { set } from 'vue' import { set } from 'vue'
import { registerPushNotifications, unregisterPushNotifications } from '../services/push/push.js' import { registerPushNotifications, unregisterPushNotifications } from '../services/push/push.js'
import oauthApi from '../services/new_api/oauth'
import { humanizeErrors } from './errors' import { humanizeErrors } from './errors'
// TODO: Unify with mergeOrAdd in statuses.js // TODO: Unify with mergeOrAdd in statuses.js
@ -368,31 +367,21 @@ const users = {
let rootState = store.rootState let rootState = store.rootState
let response = await rootState.api.backendInteractor.register(userInfo) try {
if (response.ok) { let data = await rootState.api.backendInteractor.register(userInfo)
const data = {
oauth: rootState.oauth,
instance: rootState.instance.server
}
let app = await oauthApi.getOrCreateApp(data)
let result = await oauthApi.getTokenWithCredentials({
app,
instance: data.instance,
username: userInfo.username,
password: userInfo.password
})
store.commit('signUpSuccess') store.commit('signUpSuccess')
store.commit('setToken', result.access_token) store.commit('setToken', data.access_token)
store.dispatch('loginUser', result.access_token) store.dispatch('loginUser', data.access_token)
} else { } catch (e) {
const data = await response.json() let errors = e.message
let errors = JSON.parse(data.error)
// replace ap_id with username // replace ap_id with username
if (typeof errors === 'object') {
if (errors.ap_id) { if (errors.ap_id) {
errors.username = errors.ap_id errors.username = errors.ap_id
delete errors.ap_id delete errors.ap_id
} }
errors = humanizeErrors(errors) errors = humanizeErrors(errors)
}
store.commit('signUpFailure', errors) store.commit('signUpFailure', errors)
throw Error(errors) throw Error(errors)
} }
@ -406,7 +395,7 @@ const users = {
store.dispatch('disconnectFromChat') store.dispatch('disconnectFromChat')
store.commit('setToken', false) store.commit('setToken', false)
store.dispatch('stopFetching', 'friends') store.dispatch('stopFetching', 'friends')
store.commit('setBackendInteractor', backendInteractorService()) store.commit('setBackendInteractor', backendInteractorService(store.getters.getToken()))
store.dispatch('stopFetching', 'notifications') store.dispatch('stopFetching', 'notifications')
store.commit('clearNotifications') store.commit('clearNotifications')
store.commit('resetStatuses') store.commit('resetStatuses')

View file

@ -1,6 +1,5 @@
/* eslint-env browser */ /* eslint-env browser */
const LOGIN_URL = '/api/account/verify_credentials.json' const LOGIN_URL = '/api/account/verify_credentials.json'
const REGISTRATION_URL = '/api/account/register.json'
const BG_UPDATE_URL = '/api/qvitter/update_background_image.json' const BG_UPDATE_URL = '/api/qvitter/update_background_image.json'
const EXTERNAL_PROFILE_URL = '/api/externalprofile/show.json' const EXTERNAL_PROFILE_URL = '/api/externalprofile/show.json'
const QVITTER_USER_NOTIFICATIONS_READ_URL = '/api/qvitter/statuses/notifications/read.json' const QVITTER_USER_NOTIFICATIONS_READ_URL = '/api/qvitter/statuses/notifications/read.json'
@ -17,6 +16,7 @@ const ACTIVATION_STATUS_URL = screenName => `/api/pleroma/admin/users/${screenNa
const ADMIN_USERS_URL = '/api/pleroma/admin/users' const ADMIN_USERS_URL = '/api/pleroma/admin/users'
const SUGGESTIONS_URL = '/api/v1/suggestions' const SUGGESTIONS_URL = '/api/v1/suggestions'
const MASTODON_REGISTRATION_URL = '/api/v1/accounts'
const MASTODON_USER_FAVORITES_TIMELINE_URL = '/api/v1/favourites' const MASTODON_USER_FAVORITES_TIMELINE_URL = '/api/v1/favourites'
const MASTODON_USER_NOTIFICATIONS_URL = '/api/v1/notifications' const MASTODON_USER_NOTIFICATIONS_URL = '/api/v1/notifications'
const MASTODON_FAVORITE_URL = id => `/api/v1/statuses/${id}/favourite` const MASTODON_FAVORITE_URL = id => `/api/v1/statuses/${id}/favourite`
@ -161,18 +161,28 @@ const updateProfile = ({credentials, params}) => {
// homepage // homepage
// location // location
// token // token
const register = (params) => { const register = ({ params, credentials }) => {
const form = new FormData() const { nickname, ...rest } = params
return fetch(MASTODON_REGISTRATION_URL, {
each(params, (value, key) => {
if (value) {
form.append(key, value)
}
})
return fetch(REGISTRATION_URL, {
method: 'POST', method: 'POST',
body: form headers: {
...authHeaders(credentials),
'Content-Type': 'application/json'
},
body: JSON.stringify({
nickname,
locale: 'xx_XX',
agreement: true,
...rest
})
})
.then((response) => [response.ok, response])
.then(([ok, response]) => {
if (ok) {
return response.json()
} else {
return response.json().then((error) => { throw new Error(error) })
}
}) })
} }

View file

@ -99,7 +99,7 @@ const backendInteractorService = (credentials) => {
const unpinOwnStatus = (id) => apiService.unpinOwnStatus({credentials, id}) const unpinOwnStatus = (id) => apiService.unpinOwnStatus({credentials, id})
const getCaptcha = () => apiService.getCaptcha() const getCaptcha = () => apiService.getCaptcha()
const register = (params) => apiService.register(params) const register = (params) => apiService.register({ credentials, params })
const updateAvatar = ({avatar}) => apiService.updateAvatar({credentials, avatar}) const updateAvatar = ({avatar}) => apiService.updateAvatar({credentials, avatar})
const updateBg = ({params}) => apiService.updateBg({credentials, params}) const updateBg = ({params}) => apiService.updateBg({credentials, params})
const updateBanner = ({banner}) => apiService.updateBanner({credentials, banner}) const updateBanner = ({banner}) => apiService.updateBanner({credentials, banner})

View file

@ -1,26 +1,33 @@
import { reduce } from 'lodash' import { reduce } from 'lodash'
const getOrCreateApp = ({oauth, instance}) => { const REDIRECT_URI = `${window.location.origin}/oauth-callback`
export const getOrCreateApp = ({ clientId, clientSecret, instance, commit }) => {
if (clientId && clientSecret) {
return Promise.resolve({ clientId, clientSecret })
}
const url = `${instance}/api/v1/apps` const url = `${instance}/api/v1/apps`
const form = new window.FormData() const form = new window.FormData()
form.append('client_name', `PleromaFE_${Math.random()}`) form.append('client_name', `PleromaFE_${window.___pleromafe_commit_hash}_${(new Date()).toISOString()}`)
form.append('redirect_uris', `${window.location.origin}/oauth-callback`) form.append('redirect_uris', REDIRECT_URI)
form.append('scopes', 'read write follow') form.append('scopes', 'read write follow')
return window.fetch(url, { return window.fetch(url, {
method: 'POST', method: 'POST',
body: form body: form
}).then((data) => data.json()) })
.then((data) => data.json())
.then((app) => ({ clientId: app.client_id, clientSecret: app.client_secret }))
.then((app) => commit('setClientData', app) || app)
} }
const login = (args) => {
getOrCreateApp(args).then((app) => {
args.commit('setClientData', app)
const login = ({ instance, clientId }) => {
const data = { const data = {
response_type: 'code', response_type: 'code',
client_id: app.client_id, client_id: clientId,
redirect_uri: app.redirect_uri, redirect_uri: REDIRECT_URI,
scope: 'read write follow' scope: 'read write follow'
} }
@ -34,18 +41,17 @@ const login = (args) => {
}, false) }, false)
// Do the redirect... // Do the redirect...
const url = `${args.instance}/oauth/authorize?${dataString}` const url = `${instance}/oauth/authorize?${dataString}`
window.location.href = url window.location.href = url
})
} }
const getTokenWithCredentials = ({app, instance, username, password}) => { const getTokenWithCredentials = ({ clientId, clientSecret, instance, username, password }) => {
const url = `${instance}/oauth/token` const url = `${instance}/oauth/token`
const form = new window.FormData() const form = new window.FormData()
form.append('client_id', app.client_id) form.append('client_id', clientId)
form.append('client_secret', app.client_secret) form.append('client_secret', clientSecret)
form.append('grant_type', 'password') form.append('grant_type', 'password')
form.append('username', username) form.append('username', username)
form.append('password', password) form.append('password', password)
@ -56,16 +62,32 @@ const getTokenWithCredentials = ({app, instance, username, password}) => {
}).then((data) => data.json()) }).then((data) => data.json())
} }
const getToken = ({app, instance, code}) => { const getToken = ({ clientId, clientSecret, instance, code }) => {
const url = `${instance}/oauth/token` const url = `${instance}/oauth/token`
const form = new window.FormData() const form = new window.FormData()
form.append('client_id', app.client_id) form.append('client_id', clientId)
form.append('client_secret', app.client_secret) form.append('client_secret', clientSecret)
form.append('grant_type', 'authorization_code') form.append('grant_type', 'authorization_code')
form.append('code', code) form.append('code', code)
form.append('redirect_uri', `${window.location.origin}/oauth-callback`) form.append('redirect_uri', `${window.location.origin}/oauth-callback`)
return window.fetch(url, {
method: 'POST',
body: form
})
.then((data) => data.json())
}
export const getClientToken = ({ clientId, clientSecret, instance }) => {
const url = `${instance}/oauth/token`
const form = new window.FormData()
form.append('client_id', clientId)
form.append('client_secret', clientSecret)
form.append('grant_type', 'client_credentials')
form.append('redirect_uri', `${window.location.origin}/oauth-callback`)
return window.fetch(url, { return window.fetch(url, {
method: 'POST', method: 'POST',
body: form body: form

View file

@ -5,7 +5,7 @@ const queryParams = (params) => {
} }
const headers = (store) => { const headers = (store) => {
const accessToken = store.state.oauth.token const accessToken = store.getters.getToken()
if (accessToken) { if (accessToken) {
return { 'Authorization': `Bearer ${accessToken}` } return { 'Authorization': `Bearer ${accessToken}` }
} else { } else {