Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Shot URLs are still showing up in Sentry #3769

Closed
ianb opened this issue Nov 14, 2017 · 0 comments
Closed

Shot URLs are still showing up in Sentry #3769

ianb opened this issue Nov 14, 2017 · 0 comments
Assignees
@chenba
Labels
security Security issue: can be an active issue, or related to security hygene ✿sentry✿ Something related to an error report in our Sentry error collector
Milestone
Sprint 10 (61-2) ⛅

Comments

@ianb
Copy link
Contributor

ianb commented Nov 14, 2017 •
edited
Loading

This report is an example.

I think I removed them from the add-on, and from the server, but the website is still reporting them. I think the install-raven.js handler needs some extra code for this case.
@johngruen johngruen added this to the Sprint 4 (59-3) 😯 milestone Nov 16, 2017
@chenba chenba added the ✿sentry✿ Something related to an error report in our Sentry error collector label Jan 8, 2018
@ianb ianb added the security Security issue: can be an active issue, or related to security hygene label Jan 10, 2018
@ghost ghost modified the milestones: Sprint 9 (61-1) 👋, Sprint 10 (61-2) ⛅ Apr 2, 2018
@chenba chenba self-assigned this Apr 10, 2018
chenba added a commit to chenba/screenshots that referenced this issue Apr 10, 2018
@chenba
Mask the shot url in the Raven payload. (mozilla-services#3769)
78ce81c
@chenba chenba mentioned this issue Apr 10, 2018
Merged
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Assignees

@chenba chenba

Labels
security Security issue: can be an active issue, or related to security hygene ✿sentry✿ Something related to an error report in our Sentry error collector
Projects
None yet
Milestone
Sprint 10 (61-2) ⛅
Development

No branches or pull requests
4 participants
@ianb @chenba @johngruen @marniepw