Allow only Uint8Array for screenshots

packages/core/src/js/feedback/FeedbackForm.tsx

@@ -16,7 +16,6 @@ import {
   View
 } from 'react-native';
 
-import { base64ToUint8Array } from '../utils/base64ToUint8Array';
 import { sentryLogo } from './branding';
 import { defaultConfiguration } from './defaults';
 import defaultStyles from './FeedbackForm.styles';
@@ -68,13 +67,11 @@ export class FeedbackForm extends React.Component<FeedbackFormProps, FeedbackFor
       return;
     }
 
-    const attachement = (this.state?.attachment instanceof Uint8Array)? this.state?.attachment : base64ToUint8Array(this.state?.attachment);
-
     const attachments = this.state.filename && this.state.attachment
     ? [
         {
           filename: this.state.filename,
-          data: attachement,
+          data: this.state.attachment,
         },
       ]
     : undefined;

packages/core/src/js/feedback/FeedbackForm.types.ts

@@ -156,7 +156,7 @@ export interface FeedbackCallbacks {
   /**
    * Callback when a screenshot is added
    */
-  onAddScreenshot?: (attachFile: (filename: string, base64Attachment: string | Uint8Array) => void) => void;
+  onAddScreenshot?: (attachFile: (filename: string, base64Attachment: Uint8Array) => void) => void;
 }
 
 /**

packages/core/src/js/feedback/defaults.ts

@@ -29,7 +29,7 @@ export const defaultConfiguration: Partial<FeedbackFormProps> = {
       );
     }
   },
-  onAddScreenshot: (_: (filename: string, base64Attachment: string | Uint8Array) => void) => {
+  onAddScreenshot: (_: (filename: string, base64Attachment: Uint8Array) => void) => {
     if (__DEV__) {
       Alert.alert('Development note', 'onAddScreenshot callback is not implemented.');
     }

samples/react-native/src/App.tsx

@@ -141,7 +141,37 @@ const Stack = isMobileOs
   : createStackNavigator();
 const Tab = createBottomTabNavigator();
 
-const handleChooseImage = (attachFile: (filename: string, base64Attachment: string | Uint8Array) => void): void => {
+/* eslint-disable no-bitwise */
+const base64ToUint8Array = (base64?: string): Uint8Array | undefined => {
+  if (!base64) return undefined;
+
+  const cleanedBase64 = base64.replace(/^data:.*;base64,/, ''); // Remove any prefix before the base64 string
+  const chars = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/';
+  const bytes: number[] = [];
+
+  let buffer = 0;
+  let bits = 0;
+
+  for (const char of cleanedBase64) {
+    if (char === '=') break;
+
+    const value = chars.indexOf(char); // Validate each character
+    if (value === -1) return undefined;
+
+    buffer = (buffer << 6) | value; // Shift 6 bits to the left and add the value
+    bits += 6;
+
+    if (bits >= 8) {
+      // Add a byte when we have 8 or more bits
+      bits -= 8;
+      bytes.push((buffer >> bits) & 0xff);
+    }
+  }
+
+  return new Uint8Array(bytes);
+};
+
+const handleChooseImage = (attachFile: (filename: string, base64Attachment: Uint8Array) => void): void => {
   launchImageLibrary({ mediaType: 'photo', includeBase64: true }, (response) => {
     if (response.didCancel) {
       console.log('User cancelled image picker');
@@ -150,8 +180,9 @@ const handleChooseImage = (attachFile: (filename: string, base64Attachment: stri
     } else if (response.assets && response.assets.length > 0) {
       const filename = response.assets[0].fileName;
       const base64String = response.assets[0].base64;
-      if (filename && base64String) {
-        attachFile(filename, base64String);
+      const screenShotUint8Array = base64ToUint8Array(base64String);
+      if (filename && screenShotUint8Array) {
+        attachFile(filename, screenShotUint8Array);
       }
     }
   });