Page MenuHomePhabricator
Differential D11499

[lib] Factor out innerPerformAuth for use in keyserver session recovery
ClosedPublic
Actions

Authored by ashoat on Mar 31 2024, 7:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 2, 5:39 AM
Unknown Object (File)
Tue, Apr 1, 11:06 PM
Unknown Object (File)
Tue, Apr 1, 10:19 PM
Unknown Object (File)
Tue, Apr 1, 1:00 AM
Unknown Object (File)
Thu, Mar 20, 6:35 PM
Unknown Object (File)
Feb 18 2025, 6:51 PM
Unknown Object (File)
Feb 18 2025, 6:51 PM
Unknown Object (File)
Feb 18 2025, 6:50 PM
View All 92 Files
Subscribers
None

Details

Reviewers
tomek
inka
Commits
rCOMMe1f3fd553bba: [lib] Factor out innerPerformAuth for use in keyserver session recovery
Summary

We want to be able to perform a keyserver session recovery using the new identity service-based keyserver auth.

This diff factors out a new function called innerPerformAuth, which will be used within innerPerformRecovery in the next diff.

Depends on D11498

Test Plan

Tested in combination with later diffs:

  1. Set up multi-keyserver environment following Notion doc
  2. Test native keyserver recovery via keyserverAuth by deleting a cookie in MariaDB console
  3. Hardcode incorrect userID, try same test as 2, and confirm that it fails
  4. Test web keyserver recovery in the same way as in 2
  5. Test incorrect userID on web in the same way as in 3

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
actions/
15 lines
components/
193 lines

Diff 38670

View Options

lib/actions/user-actions.js

// @flow // @flow
import invariant from 'invariant';
import * as React from 'react'; import * as React from 'react';
import { Text, View, FlatList } from 'react-native';
import {
getOwnPeerDevices,
type DeviceIDAndPlatformDetails,
} from 'lib/selectors/user-selectors.js';
import { IdentityClientContext } from 'lib/shared/identity-client-context.js';
import LinkedDevicesListItem from './linked-devices-list-item.react.js';
import type { ProfileNavigationProp } from './profile.react.js'; import type { ProfileNavigationProp } from './profile.react.js';
import type { NavigationRoute } from '../navigation/route-names.js'; import type { NavigationRoute } from '../navigation/route-names.js';
import { useSelector } from '../redux/redux-utils.js';
import { useStyles } from '../themes/colors.js';
function keyExtractor(item: DeviceIDAndPlatformDetails) {
return item.deviceID;
}
function renderDeviceListItem({
item,
index,
thisDeviceID,
}: {
+item: DeviceIDAndPlatformDetails,
+index: number,
+thisDeviceID: ?string,
...
}) {
return (
<LinkedDevicesListItem
{...item}
isPrimary={index === 0}
isThisDevice={item.deviceID === thisDeviceID}
/>
);
}
type Props = { type Props = {
+navigation: ProfileNavigationProp<'LinkedDevices'>, +navigation: ProfileNavigationProp<'LinkedDevices'>,
+route: NavigationRoute<'LinkedDevices'>, +route: NavigationRoute<'LinkedDevices'>,
}; };
// eslint-disable-next-line no-unused-vars // eslint-disable-next-line no-unused-vars
function LinkedDevices(props: Props): React.Node { function LinkedDevices(props: Props): React.Node {
return null; const styles = useStyles(unboundStyles);
const userDevicesInfos: $ReadOnlyArray<DeviceIDAndPlatformDetails> =
useSelector(getOwnPeerDevices);
const identityContext = React.useContext(IdentityClientContext);
invariant(identityContext, 'identity context not set');
const { getAuthMetadata } = identityContext;
const [thisDeviceID, setThisDeviceID] = React.useState<?string>(null);
React.useEffect(() => {
void (async () => {
const { deviceID } = await getAuthMetadata();
setThisDeviceID(deviceID);
})();
}, [getAuthMetadata]);
const separatorComponent = React.useCallback(
() => <View style={styles.separator} />,
[styles.separator],
);
const userDeviceList = React.useMemo(
() => (
<View style={styles.container}>
<Text style={styles.header}>USER DEVICES</Text>
<FlatList
data={userDevicesInfos}
renderItem={({ item, index }) =>
renderDeviceListItem({ item, index, thisDeviceID })
}
keyExtractor={keyExtractor}
contentContainerStyle={styles.deviceListContentContainer}
ItemSeparatorComponent={separatorComponent}
/>
</View>
),
[
separatorComponent,
userDevicesInfos,
styles.container,
styles.header,
styles.deviceListContentContainer,
thisDeviceID,
],
);
return userDeviceList;
} }
const unboundStyles = {
container: {
flex: 1,
backgroundColor: 'panelBackground',
paddingTop: 24,
},
header: {
color: 'panelBackgroundLabel',
fontSize: 12,
fontWeight: '400',
paddingBottom: 3,
paddingHorizontal: 24,
},
deviceListContentContainer: {
backgroundColor: 'panelForeground',
borderBottomWidth: 1,
borderColor: 'panelForegroundBorder',
borderTopWidth: 1,
marginBottom: 24,
paddingVertical: 2,
},
separator: {
backgroundColor: 'panelForegroundBorder',
height: 1,
marginHorizontal: 16,
},
};
export default LinkedDevices; export default LinkedDevices;
View Options

lib/components/keyserver-connection-handler.js

// @flow // @flow
import invariant from 'invariant';
import * as React from 'react'; import * as React from 'react';
import { Text, View, FlatList } from 'react-native';
import {
getOwnPeerDevices,
type DeviceIDAndPlatformDetails,
} from 'lib/selectors/user-selectors.js';
import { IdentityClientContext } from 'lib/shared/identity-client-context.js';
import LinkedDevicesListItem from './linked-devices-list-item.react.js';
import type { ProfileNavigationProp } from './profile.react.js'; import type { ProfileNavigationProp } from './profile.react.js';
import type { NavigationRoute } from '../navigation/route-names.js'; import type { NavigationRoute } from '../navigation/route-names.js';
import { useSelector } from '../redux/redux-utils.js';
import { useStyles } from '../themes/colors.js';
function keyExtractor(item: DeviceIDAndPlatformDetails) {
return item.deviceID;
}
function renderDeviceListItem({
item,
index,
thisDeviceID,
}: {
+item: DeviceIDAndPlatformDetails,
+index: number,
+thisDeviceID: ?string,
...
}) {
return (
<LinkedDevicesListItem
{...item}
isPrimary={index === 0}
isThisDevice={item.deviceID === thisDeviceID}
/>
);
}
type Props = { type Props = {
+navigation: ProfileNavigationProp<'LinkedDevices'>, +navigation: ProfileNavigationProp<'LinkedDevices'>,
+route: NavigationRoute<'LinkedDevices'>, +route: NavigationRoute<'LinkedDevices'>,
}; };
// eslint-disable-next-line no-unused-vars // eslint-disable-next-line no-unused-vars
function LinkedDevices(props: Props): React.Node { function LinkedDevices(props: Props): React.Node {
return null; const styles = useStyles(unboundStyles);
const userDevicesInfos: $ReadOnlyArray<DeviceIDAndPlatformDetails> =
useSelector(getOwnPeerDevices);
const identityContext = React.useContext(IdentityClientContext);
invariant(identityContext, 'identity context not set');
const { getAuthMetadata } = identityContext;
const [thisDeviceID, setThisDeviceID] = React.useState<?string>(null);
React.useEffect(() => {
void (async () => {
const { deviceID } = await getAuthMetadata();
setThisDeviceID(deviceID);
})();
}, [getAuthMetadata]);
const separatorComponent = React.useCallback(
() => <View style={styles.separator} />,
[styles.separator],
);
const userDeviceList = React.useMemo(
() => (
<View style={styles.container}>
<Text style={styles.header}>USER DEVICES</Text>
<FlatList
data={userDevicesInfos}
renderItem={({ item, index }) =>
renderDeviceListItem({ item, index, thisDeviceID })
}
keyExtractor={keyExtractor}
contentContainerStyle={styles.deviceListContentContainer}
ItemSeparatorComponent={separatorComponent}
/>
</View>
),
[
separatorComponent,
userDevicesInfos,
styles.container,
styles.header,
styles.deviceListContentContainer,
thisDeviceID,
],
);
return userDeviceList;
} }
const unboundStyles = {
container: {
flex: 1,
backgroundColor: 'panelBackground',
paddingTop: 24,
},
header: {
color: 'panelBackgroundLabel',
fontSize: 12,
fontWeight: '400',
paddingBottom: 3,
paddingHorizontal: 24,
},
deviceListContentContainer: {
backgroundColor: 'panelForeground',
borderBottomWidth: 1,
borderColor: 'panelForegroundBorder',
borderTopWidth: 1,
marginBottom: 24,
paddingVertical: 2,
},
separator: {
backgroundColor: 'panelForegroundBorder',
height: 1,
marginHorizontal: 16,
},
};
export default LinkedDevices; export default LinkedDevices;