Skip to content

Commit c648dfc

Browse files
author
Ali Abdelfattah
authored
Merge pull request #575 from Instabug/fix/upload-sourcemap
Use npx with react-native scripts
2 parents 024197a + 07f2559 commit c648dfc

File tree

5 files changed

+15
-9
lines changed

5 files changed

+15
-9
lines changed

.circleci/config.yml

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -206,13 +206,15 @@ jobs:
206206
207207
publish:
208208
macos:
209-
xcode: "10.1.0"
209+
xcode: "10.2.0"
210+
working_directory: ~
210211
steps:
211-
- checkout
212+
- checkout:
213+
path: ~/project
212214
- run: git clone https://InstabugCI:[email protected]/Instabug/Escape.git
213215
- run: cd Escape; swift build -c release -Xswiftc -static-stdlib
214216
- run: cd Escape/.build/release; cp -f Escape /usr/local/bin/escape
215-
- run: Escape react-native publish
217+
- run: cd project && Escape react-native publish
216218

217219
workflows:
218220
version: 2

__tests__/xhrNetworkInterceptor.spec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -63,7 +63,7 @@ describe('Network Interceptor', () => {
6363
const requestBody = { data: [{ item: 'first' }, { item: 'second' }] };
6464
Interceptor.enableInterception();
6565
Interceptor.setOnDoneCallback((network) => {
66-
expect(network.requestBody).toEqual(requestBody);
66+
expect(network.requestBody).toEqual(JSON.stringify(requestBody));
6767
done();
6868
})
6969
FakeRequest.open(method, url);
@@ -170,4 +170,4 @@ describe('Network Interceptor', () => {
170170
});
171171

172172

173-
});
173+
});

android/upload_sourcemap.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ else
4040
IS_HERMES=$(grep "enableHermes:" ./android/app/build.gradle -m 1)
4141
if [[ $IS_HERMES == *"true"* ]]; then
4242
#Generate android sourcemap (HERMES)
43-
react-native bundle --platform android \
43+
npx react-native bundle --platform android \
4444
--reset-cache \
4545
--entry-file index.js \
4646
--dev false \
@@ -60,7 +60,7 @@ else
6060
rm -rf index.android.bundle.map
6161
else
6262
#Generate android sourcemap
63-
react-native bundle --platform android \
63+
npx react-native bundle --platform android \
6464
--entry-file index.js \
6565
--dev false \
6666
--bundle-output ./android/main.jsbundle \

ios/upload_sourcemap.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ else
5151
echo "Instabug: Token found" "\""${INSTABUG_APP_TOKEN}"\""
5252
echo "Instabug: Generating sourcemap files..."
5353
#Generate ios sourcemap
54-
react-native bundle --platform ios \
54+
npx react-native bundle --platform ios \
5555
--entry-file index.js \
5656
--dev false \
5757
--bundle-output ./ios/main.jsbundle \

utils/XhrNetworkInterceptor.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,14 +48,18 @@ const XHRInterceptor = {
4848
if (network.requestHeaders === '') {
4949
network.requestHeaders = {};
5050
}
51-
network.requestHeaders[header] = value;
51+
network.requestHeaders[header] = typeof value === 'string' ? value : JSON.stringify(value);
5252
originalXHRSetRequestHeader.apply(this, arguments);
5353
};
5454

5555
XMLHttpRequest.prototype.send = function(data) {
5656
var cloneNetwork = JSON.parse(JSON.stringify(network));
5757
cloneNetwork.requestBody = data ? data : '';
5858

59+
if (typeof cloneNetwork.requestBody !== "string") {
60+
cloneNetwork.requestBody = JSON.stringify(cloneNetwork.requestBody);
61+
}
62+
5963
if (this.addEventListener) {
6064
this.addEventListener(
6165
'readystatechange',

0 commit comments

Comments
 (0)