Unverified Commit 20f10cf6 authored by Robert Knight's avatar Robert Knight Committed by GitHub

Merge pull request #1335 from hypothesis/use-extra-for-document-url

Record document URL as "extra" metadata rather than a tag
parents 4865ee82 fe1b5559
......@@ -50,7 +50,7 @@ function init(config) {
// via `document.referrer`. More information about the document is available
// later when frames where the "annotator" code has loaded have connected to
// the sidebar via `postMessage` RPC messages.
Sentry.setTag('document_url', document.referrer);
Sentry.setExtra('document_url', document.referrer);
}
/**
......
......@@ -10,7 +10,7 @@ describe('sidebar/util/sentry', () => {
beforeEach(() => {
fakeSentry = {
init: sinon.stub(),
setTag: sinon.stub(),
setExtra: sinon.stub(),
setUser: sinon.stub(),
};
......@@ -55,7 +55,7 @@ describe('sidebar/util/sentry', () => {
it('adds extra context to reports', () => {
sentry.init({ dsn: 'test-dsn', environment: 'dev' });
assert.calledWith(
fakeSentry.setTag,
fakeSentry.setExtra,
'document_url',
'https://example.com'
);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment