Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Track JS errors in GA #189

Merged
merged 17 commits into from
Feb 27, 2018
Merged
Show file tree
Hide file tree
Changes from 15 commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
226 changes: 226 additions & 0 deletions flow-typed/npm/raven-js_v3.17.x.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,226 @@
// flow-typed signature: e1f97cc57b871f5647a2a5a8567b0b5b
// flow-typed version: 39e54508d9/raven-js_v3.17.x/flow_>=v0.38.x

type LogLevel = 'critical' | 'error' | 'warning' | 'info' | 'debug';

type AutoBreadcrumbOptions = {
xhr?: boolean,
console?: boolean,
dom?: boolean,
location?: boolean,
};

type RavenInstrumentationOptions = {
tryCatch?: boolean,
};

type Breadcrumb = {
message?: string,
category?: string,
level?: LogLevel,
data?: any,
type?: BreadcrumbType,
};

type BreadcrumbType = 'navigation' | 'http';

type RavenOptions = {
/** The log level associated with this event. Default: error */
level?: LogLevel,

/** The name of the logger used by Sentry. Default: javascript */
logger?: string,

/** The environment of the application you are monitoring with Sentry */
environment?: string,

/** The release version of the application you are monitoring with Sentry */
release?: string,

/** The name of the server or device that the client is running on */
serverName?: string,

/** List of messages to be filtered out before being sent to Sentry. */
ignoreErrors?: (RegExp | string)[],

/** Similar to ignoreErrors, but will ignore errors from whole urls patching a regex pattern. */
ignoreUrls?: (RegExp | string)[],

/** The inverse of ignoreUrls. Only report errors from whole urls matching a regex pattern. */
whitelistUrls?: (RegExp | string)[],

/** An array of regex patterns to indicate which urls are a part of your app. */
includePaths?: (RegExp | string)[],

/** Additional data to be tagged onto the error. */
tags?: {
[id: string]: string,
},

/** set to true to get the stack trace of your message */
stacktrace?: boolean,

extra?: any,

/** In some cases you may see issues where Sentry groups multiple events together when they should be separate entities. In other cases, Sentry simply doesn’t group events together because they’re so sporadic that they never look the same. */
fingerprint?: string[],

/** A function which allows mutation of the data payload right before being sent to Sentry */
dataCallback?: (data: any) => any,

/** A callback function that allows you to apply your own filters to determine if the message should be sent to Sentry. */
shouldSendCallback?: (data: any) => boolean,

/** By default, Raven does not truncate messages. If you need to truncate characters for whatever reason, you may set this to limit the length. */
maxMessageLength?: number,

/** By default, Raven will truncate URLs as they appear in breadcrumbs and other meta interfaces to 250 characters in order to minimize bytes over the wire. This does *not* affect URLs in stack traces. */
maxUrlLength?: number,

/** Override the default HTTP data transport handler. */
transport?: (options: RavenTransportOptions) => void,

/** Allow use of private/secretKey. */
allowSecretKey?: boolean,

/** Enables/disables instrumentation of globals. */
instrument?: boolean | RavenInstrumentationOptions,

/** Enables/disables automatic collection of breadcrumbs. */
autoBreadcrumbs?: boolean | AutoBreadcrumbOptions,
};

type RavenTransportOptions = {
url: string,
data: any,
auth: {
sentry_version: string,
sentry_client: string,
sentry_key: string,
},
onSuccess: () => void,
onFailure: () => void,
};

declare module 'raven-js' {
declare type RavenPlugin = {
(raven: Raven, ...args: any[]): Raven,
};

declare class Raven {
/** Raven.js version. */
VERSION: string;

Plugins: { [id: string]: RavenPlugin };

/*
* Allow Raven to be configured as soon as it is loaded
* It uses a global RavenConfig = {dsn: '...', config: {}}
*/
afterLoad(): void;

/*
* Allow multiple versions of Raven to be installed.
* Strip Raven from the global context and returns the instance.
*/
noConflict(): this;

/** Configure Raven with a DSN and extra options */
config(dsn: string, options?: RavenOptions): this;

/*
* Installs a global window.onerror error handler
* to capture and report uncaught exceptions.
* At this point, install() is required to be called due
* to the way TraceKit is set up.
*/
install(): this;

/** Adds a plugin to Raven */
addPlugin(plugin: RavenPlugin, ...pluginArgs: any[]): this;

/*
* Wrap code within a context so Raven can capture errors
* reliably across domains that is executed immediately.
*/
context(func: Function, ...args: any[]): void;
context(options: RavenOptions, func: Function, ...args: any[]): void;

/** Wrap code within a context and returns back a new function to be executed */
wrap(func: Function): Function;
wrap(options: RavenOptions, func: Function): Function;
wrap<T: Function>(func: T): T;
wrap<T: Function>(options: RavenOptions, func: T): T;

/** Uninstalls the global error handler. */
uninstall(): this;

/** Manually capture an exception and send it over to Sentry */
captureException(ex: Error, options?: RavenOptions): this;

/** Manually send a message to Sentry */
captureMessage(msg: string, options?: RavenOptions): this;

/** Log a breadcrumb */
captureBreadcrumb(crumb: Breadcrumb): this;

/**
* Clear the user context, removing the user data that would be sent to Sentry.
*/
setUserContext(): this;

/** Set a user to be sent along with the payload. */
setUserContext(user: {
id?: string,
username?: string,
email?: string,
}): this;

/** Merge extra attributes to be sent along with the payload. */
setExtraContext(context: Object): this;

/** Merge tags to be sent along with the payload. */
setTagsContext(tags: Object): this;

/** Clear all of the context. */
clearContext(): this;

/** Get a copy of the current context. This cannot be mutated.*/
getContext(): Object;

/** Override the default HTTP data transport handler. */
setTransport(transportFunction: (options: RavenTransportOptions) => void): this;

/** Set environment of application */
setEnvironment(environment: string): this;

/** Set release version of application */
setRelease(release: string): this;

/** Get the latest raw exception that was captured by Raven.*/
lastException(): Error;

/** An event id is a globally unique id for the event that was just sent. This event id can be used to find the exact event from within Sentry. */
lastEventId(): string;

/** If you need to conditionally check if raven needs to be initialized or not, you can use the isSetup function. It will return true if Raven is already initialized. */
isSetup(): boolean;

/** Specify a function that allows mutation of the data payload right before being sent to Sentry. */
setDataCallback(data: any, orig?: any): this;

/** Specify a callback function that allows you to mutate or filter breadcrumbs when they are captured. */
setBreadcrumbCallback(data: any, orig?: any): this;

/** Specify a callback function that allows you to apply your own filters to determine if the message should be sent to Sentry. */
setShouldSendCallback(data: any, orig?: any): this;

/** Show Sentry user feedback dialog */
showReportDialog(options: Object): void;

/** Configure Raven DSN */
setDSN(dsn: string): void;
}

declare export default Raven
}
14 changes: 8 additions & 6 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
"husky": "^0.14.3",
"less-vars-to-js": "^1.2.1",
"lint-staged": "^4.0.3",
"prettier": "^1.5.3",
"prettier": "^1.10.2",
"react-app-rewire-less": "^2.1.0",
"react-app-rewired": "^1.4.0",
"react-scripts": "^1.0.11",
Expand Down Expand Up @@ -63,10 +63,11 @@
"moment": "^2.18.1",
"prop-types": "^15.5.10",
"query-string": "^5.0.0",
"raven-js": "^3.22.1",
"react": "^16.0.0",
"react-dimensions": "^1.3.0",
"react-dom": "^16.0.0",
"react-ga": "^2.2.0",
"react-ga": "^2.4.1",
"react-helmet": "^5.1.3",
"react-icons": "^2.2.7",
"react-metrics": "^2.3.2",
Expand All @@ -89,17 +90,17 @@
},
"scripts": {
"start": "react-app-rewired start",
"start:docs": "REACT_APP_DEMO=true react-scripts start",
"build": "react-app-rewired build",
"start:ga-debug":
"REACT_APP_GA_DEBUG=1 REACT_APP_VSN_STATE=$(./scripts/get-tracking-version.js) react-app-rewired start",
"build": "REACT_APP_VSN_STATE=$(./scripts/get-tracking-version.js) react-app-rewired build",
"eject": "react-scripts eject",
"test": "CI=1 react-app-rewired test --env=jsdom --color",
"test-dev": "react-app-rewired test --env=jsdom",
"coverage": "npm run test -- --coverage",
"lint": "npm run eslint && npm run prettier && npm run flow && npm run check-license",
"eslint": "eslint src",
"check-license": "./scripts/check-license.sh",
"prettier":
"prettier --write 'src/**/*.{css,js,json}' '*.{css,js,json}' && prettier --write --print-width 999999 'src/**/*.md' '*.md'",
"prettier": "prettier --write 'src/**/*.{css,js,json,md}' '*.{css,js,json,md}'",
"flow": "glow",
"precommit": "lint-staged"
},
Expand All @@ -113,6 +114,7 @@
},
"prettier": {
"printWidth": 110,
"proseWrap": "never",
"singleQuote": true,
"trailingComma": "es5"
},
Expand Down
8 changes: 0 additions & 8 deletions scripts/deploy-docs.sh

This file was deleted.

92 changes: 92 additions & 0 deletions scripts/get-tracking-version.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,92 @@
#!/usr/bin/env node

const spawnSync = require('child_process').spawnSync;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the purpose of this? Maybe a quick comment

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done.


const version = require('../package.json').version;

function cleanRemoteUrl(url) {
return url.replace(/^(.*?@|.*?\/\/)|\.git\s*$/gi, '').replace(/:/g, '/');
}

function cleanBranchNames(pointsAt) {
const branch = pointsAt.replace(/"/g, '').split('\n')[0];
const i = branch.indexOf(' ');
const objName = branch.slice(0, i);
let refName = branch.slice(i + 1);
if (refName.indexOf('detached') > -1) {
refName = '(detached)';
}
return { objName, refName };
}

function getChanged(shortstat, status) {
const rv = { hasChanged: false, files: 0, insertions: 0, deletions: 0, untracked: 0 };
const joiner = [];
const regex = /(\d+) (.)/g;
let match = regex.exec(shortstat);
while (match) {
const [, n, type] = match;
switch (type) {
case 'f':
rv.files = Number(n);
joiner.push(`${n}f`);
break;
case 'i':
rv.insertions = Number(n);
joiner.push(`+${n}`);
break;
case 'd':
rv.deletions = Number(n);
joiner.push(`-${n}`);
break;
default:
throw new Error(`Invalid diff type: ${type}`);
}
match = regex.exec(shortstat);
}
const untracked = status && status.split('\n').filter(line => line[0] === '?').length;
if (untracked) {
rv.untracked = untracked;
joiner.push(`${untracked}?`);
}
rv.pretty = joiner.join(' ');
rv.hasChanged = Boolean(joiner.length);
return rv;
}

function getVersion(cwd) {
const opts = { cwd, encoding: 'utf8' };
const url = spawnSync('git', ['remote', 'get-url', '--push', 'origin'], opts).stdout;
const branch = spawnSync(
'git',
['branch', '--points-at', 'HEAD', '--format="%(objectname:short) %(refname:short)"'],
opts
).stdout;
const shortstat = spawnSync('git', ['diff-index', '--shortstat', 'HEAD'], opts).stdout;
const status = spawnSync('git', ['status', '--porcelain', '-uall'], opts).stdout;

const { objName, refName } = cleanBranchNames(branch);
const remote = cleanRemoteUrl(url);
const joiner = [version, remote, objName];
const changed = getChanged(shortstat, status);
if (changed.hasChanged) {
joiner.push(changed.pretty);
}
joiner.push(refName);
const rv = {
version,
remote,
objName,
changed,
refName,
pretty: joiner.join(' | '),
};
return rv;
}

if (require.main === module) {
const vsn = getVersion(process.argv[2] || '.');
process.stdout.write(JSON.stringify(vsn));
} else {
module.exports = getVersion;
}
2 changes: 1 addition & 1 deletion src/components/App/Page.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ import { withRouter } from 'react-router-dom';

import TopNav from './TopNav';
import type { Config } from '../../types/config';
import { trackPageView } from '../../utils/metrics';
import { trackPageView } from '../../utils/tracking';

import './Page.css';

Expand Down
Loading