Skip to content

Commit 949c724

Browse files
lobsterkatielobsterkatie
authored
Merge branch 'master' into node-mysql
2 parents 09566ed + 22ecbcd commit 949c724

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

56 files changed

+5833
-213
lines changed

‎.craft.yml

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,18 @@ targets:
3838
onlyIfPresent: /^sentry-react-.*\.tgz$/
3939
config:
4040
canonical: 'npm:@sentry/react'
41+
- name: registry
42+
type: sdk
43+
onlyIfPresent: /^sentry-vue-.*\.tgz$/
44+
config:
45+
canonical: 'npm:@sentry/vue'
4146
- name: registry
4247
type: sdk
4348
onlyIfPresent: /^sentry-gatsby-.*\.tgz$/
4449
config:
4550
canonical: 'npm:@sentry/gatsby'
51+
- name: registry
52+
type: sdk
53+
onlyIfPresent: /^sentry-angular-.*\.tgz$/
54+
config:
55+
canonical: 'npm:@sentry/angular'

‎CHANGELOG.md

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,15 @@
44

55
- "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
66

7+
## 5.28.0
8+
9+
- [browser] fix: Handle expo file dir stack frames (#3070)
10+
- [vue] feat: @sentry/vue (#2953)
11+
- [node] ref: Revamp express route info extraction (#3084)
12+
- [browser] fix: Dont append dsn twice to report dialog calls (#3079)
13+
- [ember] fix: Use correct import from `@sentry/browser` (#3077)
14+
- [node] ref: Express integration span name change and path unification (#3078)
15+
716
## 5.27.6
817

918
- [hub] fix: Don't invoke scope updates in scope listeners

‎README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,11 @@ package. Please refer to the README and instructions of those SDKs for more deta
4141
including integrations for React, Angular, Ember, Vue and Backbone
4242
- [`@sentry/node`](https://github.com/getsentry/sentry-javascript/tree/master/packages/node): SDK for Node, including
4343
integrations for Express, Koa, Loopback, Sails and Connect
44+
- [`@sentry/angular`](https://github.com/getsentry/sentry-javascript/tree/master/packages/angular): SDK for Angular
45+
- [`@sentry/react`](https://github.com/getsentry/sentry-javascript/tree/master/packages/react): SDK for ReactJS
46+
- [`@sentry/ember`](https://github.com/getsentry/sentry-javascript/tree/master/packages/ember): SDK for Ember
47+
- [`@sentry/vue`](https://github.com/getsentry/sentry-javascript/tree/master/packages/vue): SDK for Vue.js
48+
- [`@sentry/gatsby`](https://github.com/getsentry/sentry-javascript/tree/master/packages/gatsby): SDK for Gatsby
4449
- [`@sentry/react-native`](https://github.com/getsentry/sentry-react-native): SDK for React Native with support for native crashes
4550
- [`@sentry/integrations`](https://github.com/getsentry/sentry-javascript/tree/master/packages/integrations): Pluggable
4651
integrations that can be used to enhance JS SDKs

‎lerna.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"lerna": "3.4.0",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"packages": "packages/*",
55
"npmClient": "yarn",
66
"useWorkspaces": true

‎package.json

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
},
1717
"volta": {
1818
"node": "10.18.1",
19-
"yarn": "1.13.0"
19+
"yarn": "1.22.5"
2020
},
2121
"workspaces": [
2222
"packages/angular",
@@ -35,7 +35,8 @@
3535
"packages/tracing",
3636
"packages/types",
3737
"packages/typescript",
38-
"packages/utils"
38+
"packages/utils",
39+
"packages/vue"
3940
],
4041
"devDependencies": {
4142
"@google-cloud/storage": "^2.5.0",
@@ -44,7 +45,7 @@
4445
"@types/chai": "^4.1.3",
4546
"@types/jest": "^24.0.11",
4647
"@types/mocha": "^5.2.0",
47-
"@types/node": "^11.13.7",
48+
"@types/node": "~10.17.0",
4849
"@types/sinon": "^7.0.11",
4950
"chai": "^4.1.2",
5051
"codecov": "^3.6.5",

‎packages/angular/package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry/angular",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Offical Sentry SDK for Angular",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/angular",
@@ -21,17 +21,17 @@
2121
"@angular/router": "^10.0.3"
2222
},
2323
"dependencies": {
24-
"@sentry/browser": "5.27.6",
25-
"@sentry/types": "5.27.6",
26-
"@sentry/utils": "5.27.6",
24+
"@sentry/browser": "5.28.0",
25+
"@sentry/types": "5.28.0",
26+
"@sentry/utils": "5.28.0",
2727
"rxjs": "^6.6.0",
2828
"tslib": "^1.9.3"
2929
},
3030
"devDependencies": {
3131
"@angular/common": "^10.0.3",
3232
"@angular/core": "^10.0.3",
3333
"@angular/router": "^10.0.3",
34-
"@sentry-internal/eslint-config-sdk": "5.27.6",
34+
"@sentry-internal/eslint-config-sdk": "5.28.0",
3535
"eslint": "7.6.0",
3636
"npm-run-all": "^4.1.2",
3737
"prettier": "1.19.0",

‎packages/browser/package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry/browser",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Official Sentry SDK for browsers",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/browser",
@@ -16,13 +16,13 @@
1616
"access": "public"
1717
},
1818
"dependencies": {
19-
"@sentry/core": "5.27.6",
20-
"@sentry/types": "5.27.6",
21-
"@sentry/utils": "5.27.6",
19+
"@sentry/core": "5.28.0",
20+
"@sentry/types": "5.28.0",
21+
"@sentry/utils": "5.28.0",
2222
"tslib": "^1.9.3"
2323
},
2424
"devDependencies": {
25-
"@sentry-internal/eslint-config-sdk": "5.27.6",
25+
"@sentry-internal/eslint-config-sdk": "5.28.0",
2626
"@types/eslint": "^7.2.0",
2727
"@types/md5": "2.1.33",
2828
"btoa": "^1.2.1",

‎packages/browser/src/tracekit.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ const chrome = /^\s*at (?:(.*?) ?\()?((?:file|https?|blob|chrome-extension|addre
4646
// gecko regex: `(?:bundle|\d+\.js)`: `bundle` is for react native, `\d+\.js` also but specifically for ram bundles because it
4747
// generates filenames without a prefix like `file://` the filenames in the stacktrace are just 42.js
4848
// We need this specific case for now because we want no other regex to match.
49-
const gecko = /^\s*(.*?)(?:\((.*?)\))?(?:^|@)?((?:file|https?|blob|chrome|webpack|resource|moz-extension|capacitor).*?:\/.*?|\[native code\]|[^@]*(?:bundle|\d+\.js))(?::(\d+))?(?::(\d+))?\s*$/i;
49+
const gecko = /^\s*(.*?)(?:\((.*?)\))?(?:^|@)?((?:file|https?|blob|chrome|webpack|resource|moz-extension|capacitor).*?:\/.*?|\[native code\]|[^@]*(?:bundle|\d+\.js)|\/[\w\-. /=]+)(?::(\d+))?(?::(\d+))?\s*$/i;
5050
const winjs = /^\s*at (?:((?:\[object object\])?.+) )?\(?((?:file|ms-appx|https?|webpack|blob):.*?):(\d+)(?::(\d+))?\)?\s*$/i;
5151
const geckoEval = /(\S+) line (\d+)(?: > eval line \d+)* > eval/i;
5252
const chromeEval = /\((\S*)(?::(\d+))(?::(\d+))\)/;

‎packages/browser/src/transports/base.ts

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ export abstract class BaseTransport implements Transport {
6767
* https://developer.mozilla.org/en-US/docs/Web/API/Headers/get
6868
*/
6969
const limited = this._handleRateLimit(headers);
70-
if (limited) logger.warn(`Too many requests, backing off till: ${this._disabledUntil(requestType)}`);
70+
if (limited) logger.warn(`Too many requests, backing off until: ${this._disabledUntil(requestType)}`);
7171

7272
if (status === Status.Success) {
7373
resolve({ status });
@@ -100,6 +100,16 @@ export abstract class BaseTransport implements Transport {
100100
const raHeader = headers['retry-after'];
101101

102102
if (rlHeader) {
103+
// rate limit headers are of the form
104+
// <header>,<header>,..
105+
// where each <header> is of the form
106+
// <retry_after>: <categories>: <scope>: <reason_code>
107+
// where
108+
// <retry_after> is a delay in ms
109+
// <categories> is the event type(s) (error, transaction, etc) being rate limited and is of the form
110+
// <category>;<category>;...
111+
// <scope> is what's being limited (org, project, or key) - ignored by SDK
112+
// <reason_code> is an arbitrary string like "org_quota" - ignored by SDK
103113
for (const limit of rlHeader.trim().split(',')) {
104114
const parameters = limit.split(':', 2);
105115
const headerDelay = parseInt(parameters[0], 10);

‎packages/browser/src/transports/fetch.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,13 @@ export class FetchTransport extends BaseTransport {
6161
'x-sentry-rate-limits': response.headers.get('X-Sentry-Rate-Limits'),
6262
'retry-after': response.headers.get('Retry-After'),
6363
};
64-
this._handleResponse({ requestType: sentryRequest.type, response, headers, resolve, reject });
64+
this._handleResponse({
65+
requestType: sentryRequest.type,
66+
response,
67+
headers,
68+
resolve,
69+
reject,
70+
});
6571
})
6672
.catch(reject);
6773
}),

‎packages/browser/src/version.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
export const SDK_NAME = 'sentry.javascript.browser';
2-
export const SDK_VERSION = '5.27.6';
2+
export const SDK_VERSION = '5.28.0';

‎packages/browser/test/unit/tracekit/custom.test.ts

Lines changed: 54 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,60 @@ describe('Tracekit - Custom Tests', () => {
4949
]);
5050
});
5151

52+
it('should parse exceptions for react-native Expo bundles', () => {
53+
const REACT_NATIVE_EXPO_EXCEPTION = {
54+
message: 'Test Error Expo',
55+
name: 'Error',
56+
stack: `onPress@/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3:595:658
57+
value@/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3:221:7656
58+
onResponderRelease@/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3:221:5666
59+
p@/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3:96:385
60+
forEach@[native code]`,
61+
};
62+
const stacktrace = computeStackTrace(REACT_NATIVE_EXPO_EXCEPTION);
63+
expect(stacktrace.stack).deep.equal([
64+
{
65+
url:
66+
'/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3',
67+
func: 'onPress',
68+
args: [],
69+
line: 595,
70+
column: 658,
71+
},
72+
{
73+
url:
74+
'/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3',
75+
func: 'value',
76+
args: [],
77+
line: 221,
78+
column: 7656,
79+
},
80+
{
81+
url:
82+
'/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3',
83+
func: 'onResponderRelease',
84+
args: [],
85+
line: 221,
86+
column: 5666,
87+
},
88+
{
89+
url:
90+
'/data/user/0/com.sentrytest/files/.expo-internal/bundle-613EDD44F3305B9D75D4679663900F2BCDDDC326F247CA3202A3A4219FD412D3',
91+
func: 'p',
92+
args: [],
93+
line: 96,
94+
column: 385,
95+
},
96+
{
97+
url: '[native code]',
98+
func: 'forEach',
99+
args: [],
100+
line: null,
101+
column: null,
102+
},
103+
]);
104+
});
105+
52106
describe('should parse exceptions with native code frames', () => {
53107
it('in Chrome 73', () => {
54108
const CHROME73_NATIVE_CODE_EXCEPTION = {

‎packages/core/package.json

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry/core",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Base implementation for all Sentry JavaScript SDKs",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/core",
@@ -16,14 +16,14 @@
1616
"access": "public"
1717
},
1818
"dependencies": {
19-
"@sentry/hub": "5.27.6",
20-
"@sentry/minimal": "5.27.6",
21-
"@sentry/types": "5.27.6",
22-
"@sentry/utils": "5.27.6",
19+
"@sentry/hub": "5.28.0",
20+
"@sentry/minimal": "5.28.0",
21+
"@sentry/types": "5.28.0",
22+
"@sentry/utils": "5.28.0",
2323
"tslib": "^1.9.3"
2424
},
2525
"devDependencies": {
26-
"@sentry-internal/eslint-config-sdk": "5.27.6",
26+
"@sentry-internal/eslint-config-sdk": "5.28.0",
2727
"eslint": "7.6.0",
2828
"jest": "^24.7.1",
2929
"npm-run-all": "^4.1.2",

‎packages/ember/package.json

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry/ember",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Offical Sentry SDK for Ember.js",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/ember",
@@ -32,10 +32,10 @@
3232
},
3333
"dependencies": {
3434
"@embroider/macros": ">=0.25.0",
35-
"@sentry/browser": "5.27.6",
36-
"@sentry/tracing": "5.27.6",
37-
"@sentry/types": "5.27.6",
38-
"@sentry/utils": "5.27.6",
35+
"@sentry/browser": "5.28.0",
36+
"@sentry/tracing": "5.28.0",
37+
"@sentry/types": "5.28.0",
38+
"@sentry/utils": "5.28.0",
3939
"ember-auto-import": "^1.6.0",
4040
"ember-cli-babel": "^7.20.5",
4141
"ember-cli-htmlbars": "^5.1.2",
@@ -46,7 +46,7 @@
4646
"@ember/optional-features": "^1.3.0",
4747
"@glimmer/component": "^1.0.0",
4848
"@glimmer/tracking": "^1.0.0",
49-
"@sentry-internal/eslint-config-sdk": "5.27.6",
49+
"@sentry-internal/eslint-config-sdk": "5.28.0",
5050
"@types/ember": "^3.16.0",
5151
"@types/ember-qunit": "^3.4.9",
5252
"@types/ember__test-helpers": "^1.7.0",
@@ -92,6 +92,6 @@
9292
},
9393
"volta": {
9494
"node": "12.18.3",
95-
"yarn": "1.22.4"
95+
"yarn": "1.22.5"
9696
}
9797
}

‎packages/eslint-config-sdk/package.json

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry-internal/eslint-config-sdk",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Official Sentry SDK eslint config",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/eslint-config-sdk",
@@ -19,8 +19,8 @@
1919
"access": "public"
2020
},
2121
"dependencies": {
22-
"@sentry-internal/eslint-plugin-sdk": "5.27.6",
23-
"@sentry-internal/typescript": "5.27.6",
22+
"@sentry-internal/eslint-plugin-sdk": "5.28.0",
23+
"@sentry-internal/typescript": "5.28.0",
2424
"@typescript-eslint/eslint-plugin": "^3.9.0",
2525
"@typescript-eslint/parser": "^3.9.0",
2626
"eslint-config-prettier": "^6.11.0",

‎packages/eslint-plugin-sdk/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry-internal/eslint-plugin-sdk",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Official Sentry SDK eslint plugin",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/eslint-plugin-sdk",

‎packages/gatsby/package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry/gatsby",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Offical Sentry SDK for Gatsby.js",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/gatsby",
@@ -26,14 +26,14 @@
2626
"access": "public"
2727
},
2828
"dependencies": {
29-
"@sentry/react": "5.27.6",
30-
"@sentry/tracing": "5.27.6"
29+
"@sentry/react": "5.28.0",
30+
"@sentry/tracing": "5.28.0"
3131
},
3232
"peerDependencies": {
3333
"gatsby": "*"
3434
},
3535
"devDependencies": {
36-
"@sentry-internal/eslint-config-sdk": "5.27.6",
36+
"@sentry-internal/eslint-config-sdk": "5.28.0",
3737
"@testing-library/react": "^10.4.9",
3838
"eslint": "7.6.0",
3939
"jest": "^24.7.1",

‎packages/hub/package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@sentry/hub",
3-
"version": "5.27.6",
3+
"version": "5.28.0",
44
"description": "Sentry hub which handles global state managment.",
55
"repository": "git://github.com/getsentry/sentry-javascript.git",
66
"homepage": "https://github.com/getsentry/sentry-javascript/tree/master/packages/hub",
@@ -16,12 +16,12 @@
1616
"access": "public"
1717
},
1818
"dependencies": {
19-
"@sentry/types": "5.27.6",
20-
"@sentry/utils": "5.27.6",
19+
"@sentry/types": "5.28.0",
20+
"@sentry/utils": "5.28.0",
2121
"tslib": "^1.9.3"
2222
},
2323
"devDependencies": {
24-
"@sentry-internal/eslint-config-sdk": "5.27.6",
24+
"@sentry-internal/eslint-config-sdk": "5.28.0",
2525
"eslint": "7.6.0",
2626
"jest": "^24.7.1",
2727
"npm-run-all": "^4.1.2",

0 commit comments

Comments
 (0)