Export logs from filesystem instead of store. Add isMobile conditional to store and dont show share button on desktop.

Author: wbobeirne

src/action/index-mobile.js

@@ -43,11 +43,12 @@ import AtplAction from './autopilot';
 //
 
 store.init(); // initialize computed values
+store.isMobile = true;
 
 export const db = new AppStorage(store, AsyncStorage);
 export const grpc = new GrpcAction(store, NativeModules, NativeEventEmitter);
 export const ipc = new IpcAction(grpc);
-export const log = new LogAction(store, ipc, false);
+export const log = new LogAction(store, ipc, false, RNFS);
 export const nav = new NavAction(store, NavigationActions, StackActions);
 export const notify = new NotificationAction(store, nav);
 export const wallet = new WalletAction(store, grpc, db, nav, notify, RNFS);

src/action/log.js

@@ -10,6 +10,7 @@ import { MAX_LOG_LENGTH } from '../config';
 let _store;
 let _ipc;
 let _printErrObj;
+let _FS;
 
 /**
  * Log an info event e.g. when something relevant but non-critical happens.
@@ -49,6 +50,29 @@ export function error(...args) {
   _ipc && _ipc.send('log-error', null, args);
 }
 
+/**
+ * Gets the location of the LND log file as a string.
+ * @param {string} network The network the user's on
+ * @return {string}
+ */
+export function getLogPath(network) {
+  const lndDir = _FS.DocumentDirectoryPath;
+  return `${lndDir}/logs/bitcoin/${network}/lnd.log`;
+}
+
+/**
+ * Retrieves the entire LND log file as a string.
+ * @return {Promise}
+ */
+export function getLogs() {
+  if (!_FS) {
+    return Promise.reject(
+      'Cannot get logs when no filesystem is available in action/log.js'
+    );
+  }
+  return _FS.readFile(getLogPath(_store.network), 'utf8');
+}
+
 function pushLogs(message) {
   if (!_store) return;
   _store.logs += '\n' + message.replace(/\s+$/, '');
@@ -59,10 +83,11 @@ function pushLogs(message) {
 }
 
 class LogAction {
-  constructor(store, ipc, printErrObj = true) {
+  constructor(store, ipc, printErrObj = true, FS) {
     _store = store;
     _ipc = ipc;
     _printErrObj = printErrObj;
+    _FS = FS;
     _ipc.listen('logs', (event, message) => pushLogs(message));
     _ipc.send('logs-ready', null, true);
   }

src/store.js

@@ -19,6 +19,7 @@ import { DEFAULT_ROUTE, DEFAULT_UNIT, DEFAULT_FIAT } from './config';
 export class Store {
   constructor() {
     extendObservable(this, {
+      isMobile: false, // Mobile app or desktop electron
       navReady: false, // the navigator on mobile is ready
       loaded: false, // Is persistent data loaded
       unlockerReady: false, // Is wallet unlocker running

src/view/cli.js

@@ -5,9 +5,10 @@ import PropTypes from 'prop-types';
 import { SplitBackground } from '../component/background';
 import { Header, Title } from '../component/header';
 import Text from '../component/text';
-import { BackButton, ShareButton } from '../component/button';
+import { BackButton, ShareButton, Button } from '../component/button';
 import { createStyles, maxWidth } from '../component/media-query';
 import { color, font, breakWidth } from '../component/style';
+import { getLogs, error } from '../action/log';
 
 //
 // CLI View
@@ -27,20 +28,26 @@ class CLIView extends Component {
         <Header separator style={styles.header}>
           <BackButton onPress={() => nav.goSettings()} />
           <Title title="Logs" />
-          <ShareButton onPress={() => this.shareLogs()} />
+          {store.isMobile ? (
+            <ShareButton onPress={() => this.shareLogs()} />
+          ) : (
+            <Button onPress={() => {}} />
+          )}
         </Header>
         <LogOutput logs={store.logs} />
       </SplitBackground>
     );
   }
 
-  shareLogs() {
+  async shareLogs() {
     try {
-      Share.share({
-        message: this.props.store.logs,
+      const logs = await getLogs();
+      await Share.share({
+        title: 'Lightning App logs',
+        message: logs,
       });
     } catch (err) {
-      console.error(err);
+      error(err);
     }
   }
 }