Refactor auth to support multiple accounts

merge-requests/451/head
Alex Gleason 2021-03-23 19:06:55 -05:00
rodzic 8e386ddfd4
commit 0162eac662
Nie znaleziono w bazie danych klucza dla tego podpisu
ID klucza GPG: 7211D1F99744FBB7
9 zmienionych plików z 162 dodań i 20 usunięć

Wyświetl plik

@ -1,11 +1,17 @@
import api from '../api';
import snackbar from 'soapbox/actions/snackbar';
export const SWITCH_ACCOUNT = 'SWITCH_ACCOUNT';
export const AUTH_APP_CREATED = 'AUTH_APP_CREATED';
export const AUTH_APP_AUTHORIZED = 'AUTH_APP_AUTHORIZED';
export const AUTH_LOGGED_IN = 'AUTH_LOGGED_IN';
export const AUTH_LOGGED_OUT = 'AUTH_LOGGED_OUT';
export const VERIFY_CREDENTIALS_REQUEST = 'VERIFY_CREDENTIALS_REQUEST';
export const VERIFY_CREDENTIALS_SUCCESS = 'VERIFY_CREDENTIALS_SUCCESS';
export const VERIFY_CREDENTIALS_FAIL = 'VERIFY_CREDENTIALS_FAIL';
export const AUTH_REGISTER_REQUEST = 'AUTH_REGISTER_REQUEST';
export const AUTH_REGISTER_SUCCESS = 'AUTH_REGISTER_SUCCESS';
export const AUTH_REGISTER_FAIL = 'AUTH_REGISTER_FAIL';
@ -127,6 +133,27 @@ export function otpVerify(code, mfa_token) {
};
}
export function verifyCredentials(token) {
return (dispatch, getState) => {
dispatch({ type: VERIFY_CREDENTIALS_REQUEST });
const request = {
method: 'get',
url: '/api/v1/accounts/verify_credentials',
headers: {
'Authorization': `Bearer ${token.get('access_token')}`,
},
};
return api(getState).request(request).then(({ data: account }) => {
dispatch({ type: VERIFY_CREDENTIALS_SUCCESS, token, account });
return account;
}).catch(error => {
dispatch({ type: VERIFY_CREDENTIALS_FAIL, token, error });
});
};
}
export function logIn(username, password) {
return (dispatch, getState) => {
return dispatch(createAppAndToken()).then(() => {
@ -161,6 +188,10 @@ export function logOut() {
};
}
export function switchAccount(accountId) {
return { type: SWITCH_ACCOUNT, accountId };
}
export function register(params) {
return (dispatch, getState) => {
params.fullname = params.username;

Wyświetl plik

@ -1,5 +1,7 @@
import api from '../api';
import { importFetchedAccount } from './importer';
import { List as ImmutableList } from 'immutable';
import { verifyCredentials } from './auth';
export const ME_FETCH_REQUEST = 'ME_FETCH_REQUEST';
export const ME_FETCH_SUCCESS = 'ME_FETCH_SUCCESS';
@ -10,23 +12,25 @@ export const ME_PATCH_REQUEST = 'ME_PATCH_REQUEST';
export const ME_PATCH_SUCCESS = 'ME_PATCH_SUCCESS';
export const ME_PATCH_FAIL = 'ME_PATCH_FAIL';
const hasToken = getState => getState().hasIn(['auth', 'user', 'access_token']);
const noOp = () => new Promise(f => f());
export function fetchMe() {
return (dispatch, getState) => {
const state = getState();
if (!hasToken(getState)) {
const me = state.getIn(['auth', 'me']);
const token = state.getIn(['auth', 'users', me]);
if (!token) {
dispatch({ type: ME_FETCH_SKIP }); return noOp();
};
dispatch(fetchMeRequest());
return api(getState).get('/api/v1/accounts/verify_credentials').then(response => {
dispatch(fetchMeSuccess(response.data));
return dispatch(verifyCredentials(token)).then(account => {
dispatch(fetchMeSuccess(account));
}).catch(error => {
dispatch(fetchMeFail(error));
});
});;
};
}

Wyświetl plik

@ -9,8 +9,15 @@ export const getLinks = response => {
return LinkHeader.parse(value);
};
const getToken = (getState, authType) =>
getState().getIn(['auth', authType, 'access_token']);
const getToken = (getState, authType) => {
const state = getState();
if (authType === 'app') {
return state.getIn(['auth', 'app', 'access_token']);
} else {
const me = state.get('me');
return state.getIn(['auth', 'users', me, 'access_token']);
}
};
export default (getState, authType = 'user') => {
const accessToken = getToken(getState, authType);

Wyświetl plik

@ -0,0 +1,91 @@
import React from 'react';
import { connect } from 'react-redux';
// import { openModal } from '../../../actions/modal';
import PropTypes from 'prop-types';
import ImmutablePropTypes from 'react-immutable-proptypes';
import DropdownMenuContainer from '../../../containers/dropdown_menu_container';
import { isStaff } from 'soapbox/utils/accounts';
import { defineMessages, injectIntl } from 'react-intl';
import { logOut, switchAccount } from 'soapbox/actions/auth';
import { Map as ImmutableMap, List as ImmutableList } from 'immutable';
const messages = defineMessages({
switch: { id: 'profile_dropdown.switch_account', defaultMessage: 'Switch to @{acct}' },
logout: { id: 'profile_dropdown.logout', defaultMessage: 'Log out @{acct}' },
});
const mapStateToProps = state => {
const me = state.get('me');
const otherAccounts =
state
.getIn(['auth', 'users'])
.keySeq()
.reduce((list, id) => {
if (id === me) return list;
const account = state.getIn(['accounts', id]) || ImmutableMap({ id: id, acct: id });
return list.push(account);
}, ImmutableList());
return {
account: state.getIn(['accounts', me]),
otherAccounts,
isStaff: isStaff(state.getIn(['accounts', me])),
};
};
class ProfileDropdown extends React.PureComponent {
static propTypes = {
intl: PropTypes.object.isRequired,
dispatch: PropTypes.func.isRequired,
size: PropTypes.number,
account: ImmutablePropTypes.map,
otherAccounts: ImmutablePropTypes.list,
isStaff: PropTypes.bool.isRequired,
};
static defaultProps = {
isStaff: false,
}
handleLogOut = e => {
this.props.dispatch(logOut());
e.preventDefault();
};
handleSwitchAccount = account => {
return e => {
this.props.dispatch(switchAccount(account.get('id')));
e.preventDefault();
};
}
render() {
const { intl, account, otherAccounts } = this.props;
const size = this.props.size || 16;
let menu = [];
otherAccounts.forEach(account => {
menu.push({ text: intl.formatMessage(messages.switch, { acct: account.get('acct') }), action: this.handleSwitchAccount(account) });
});
if (otherAccounts.size > 0) {
menu.push(null);
}
menu.push({ text: intl.formatMessage(messages.logout, { acct: account.get('acct') }), to: '/auth/sign_out', action: this.handleLogOut });
return (
<div className='compose__action-bar' style={{ 'marginTop':'-6px' }}>
<div className='compose__action-bar-dropdown'>
<DropdownMenuContainer items={menu} icon='chevron-down' size={size} direction='right' />
</div>
</div>
);
}
}
export default injectIntl(connect(mapStateToProps)(ProfileDropdown));

Wyświetl plik

@ -8,7 +8,7 @@ import classNames from 'classnames';
import IconWithCounter from 'soapbox/components/icon_with_counter';
import SearchContainer from 'soapbox/features/compose/containers/search_container';
import Avatar from '../../../components/avatar';
import ActionBar from 'soapbox/features/compose/components/action_bar';
import ProfileDropdown from './profile_dropdown';
import { openModal } from '../../../actions/modal';
import { openSidebar } from '../../../actions/sidebar';
import Icon from '../../../components/icon';
@ -126,7 +126,7 @@ class TabsBar extends React.PureComponent {
<div className='tabs-bar__profile'>
<Avatar account={account} />
<button className='tabs-bar__sidebar-btn' onClick={onOpenSidebar} />
<ActionBar account={account} size={34} />
<ProfileDropdown account={account} size={34} />
</div>
<button className='tabs-bar__button-compose button' onClick={onOpenCompose} aria-label={intl.formatMessage(messages.post)}>
<span>{intl.formatMessage(messages.post)}</span>

Wyświetl plik

@ -5,12 +5,14 @@ import {
AUTH_LOGGED_OUT,
FETCH_TOKENS_SUCCESS,
REVOKE_TOKEN_SUCCESS,
SWITCH_ACCOUNT,
} from '../actions/auth';
import { Map as ImmutableMap, List as ImmutableList, fromJS } from 'immutable';
const initialState = ImmutableMap({
app: ImmutableMap(JSON.parse(localStorage.getItem('soapbox:auth:app'))),
user: ImmutableMap(JSON.parse(localStorage.getItem('soapbox:auth:user'))),
users: fromJS(JSON.parse(localStorage.getItem('soapbox:auth:users'))),
me: localStorage.getItem('soapbox:auth:me'),
tokens: ImmutableList(),
});
@ -24,7 +26,6 @@ export default function auth(state = initialState, action) {
localStorage.setItem('soapbox:auth:app', JSON.stringify(merged)); // TODO: Better persistence
return state.set('app', merged);
case AUTH_LOGGED_IN:
localStorage.setItem('soapbox:auth:user', JSON.stringify(action.user)); // TODO: Better persistence
return state.set('user', ImmutableMap(action.user));
case AUTH_LOGGED_OUT:
localStorage.removeItem('soapbox:auth:user');
@ -34,6 +35,10 @@ export default function auth(state = initialState, action) {
case REVOKE_TOKEN_SUCCESS:
const idx = state.get('tokens').findIndex(t => t.get('id') === action.id);
return state.deleteIn(['tokens', idx]);
case SWITCH_ACCOUNT:
localStorage.setItem('soapbox:auth:me', action.accountId);
location.reload();
return state;
default:
return state;
}

Wyświetl plik

@ -16,7 +16,6 @@ export default function me(state = initialState, action) {
case ME_FETCH_FAIL:
case ME_FETCH_SKIP:
case AUTH_LOGGED_OUT:
localStorage.removeItem('soapbox:auth:user');
return false;
default:
return state;

Wyświetl plik

@ -1,7 +1,8 @@
'use strict';
import { ME_FETCH_SUCCESS, ME_PATCH_SUCCESS } from 'soapbox/actions/me';
import { Map as ImmutableMap, fromJS } from 'immutable';
import { VERIFY_CREDENTIALS_SUCCESS } from 'soapbox/actions/auth';
import { Map as ImmutableMap, OrderedSet as ImmutableOrderedSet, fromJS } from 'immutable';
const initialState = ImmutableMap();
@ -10,11 +11,16 @@ export default function meta(state = initialState, action) {
case ME_FETCH_SUCCESS:
case ME_PATCH_SUCCESS:
const me = fromJS(action.me);
if (me.has('pleroma')) {
const pleroPrefs = me.get('pleroma').delete('settings_store');
return state.mergeIn(['pleroma'], pleroPrefs);
}
return state;
return state.withMutations(state => {
state.set('me', me.get('id'));
state.update('users', ImmutableOrderedSet(), v => v.add(me.get('id')));
if (me.has('pleroma')) {
const pleroPrefs = me.get('pleroma').delete('settings_store');
state.mergeIn(['pleroma'], pleroPrefs);
}
});
case VERIFY_CREDENTIALS_SUCCESS:
return state.update('users', ImmutableOrderedSet(), v => v.add(action.account.id));
default:
return state;
}

Wyświetl plik

@ -62,7 +62,6 @@
text-decoration: none;
text-overflow: ellipsis;
white-space: nowrap;
text-transform: capitalize;
color: var(--primary-text-color);
&:focus,