You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
jsdeliver is definitely failing to do the right thing. They're missing a file in reference in the sourcemap. At least, that's what I'm assuming the issue is.
Either way, I'm not sure how this is an issue. What is cdn.jsdelivr.net/g/bundle.map and how does that relate? Typically these errors are just fine. It doesn't break anything, you just can't browse the unminified source in your debug console. But it won't affect any production applications.
Also worth noting, that I didn't even know jsdeliver existed until just now. We don't maintain this and this is the first I've heard of it existing.
The only issue I can see is that they're missing the file https://cdn.jsdelivr.net/raven/3.3.0/raven.js to go with https://cdn.jsdelivr.net/raven/3.3.0/raven.min.js.map
Thanks for looking into it @mattrobenolt! JSDelivr is a popular CDN. I assumed that you guys had uploaded Raven there but I realize now that I got confused about the contributors while hopping between their repo and here. The error doesn't break anything, it's just annoying in our console :)
@wearhere see jsdelivr/jsdelivr#13078 which has been open for over 3 months now. Again, there's nothing more we can do here on our side since we don't control JSDelivr.
Oh I know, I wasn't complaining. Thanks again for your research here. I was just explaining why someone might choose to use JSDelivr vs. the official CDN, I agree it would be nice to follow the official recommendation.
Activity
mattrobenolt commentedon Aug 4, 2016
jsdeliver is definitely failing to do the right thing. They're missing a file in reference in the sourcemap. At least, that's what I'm assuming the issue is.
Either way, I'm not sure how this is an issue. What is
cdn.jsdelivr.net/g/bundle.mapand how does that relate? Typically these errors are just fine. It doesn't break anything, you just can't browse the unminified source in your debug console. But it won't affect any production applications.Also worth noting, that I didn't even know jsdeliver existed until just now. We don't maintain this and this is the first I've heard of it existing.
The only issue I can see is that they're missing the file
https://cdn.jsdelivr.net/raven/3.3.0/raven.jsto go withhttps://cdn.jsdelivr.net/raven/3.3.0/raven.min.js.mapmattrobenolt commentedon Aug 4, 2016
I've opened an issue up over at JSDeliver to see what they can do about it.
wearhere commentedon Aug 4, 2016
Thanks for looking into it @mattrobenolt! JSDelivr is a popular CDN. I assumed that you guys had uploaded Raven there but I realize now that I got confused about the contributors while hopping between their repo and here. The error doesn't break anything, it's just annoying in our console :)
benvinegar commentedon Nov 12, 2016
Since we don't control JSDeliver, I'm going to close this issue.
We strongly recommend using the official Raven.js CDN.
wearhere commentedon Nov 12, 2016
Fair enough @benvinegar. The only thing that's nice about JSDelivr is that you can bundle different scripts together. For instance
fetches jQuery, Underscore, and Raven altogether. This saves on bandwidth by compressing the scripts together.
mattrobenolt commentedon Nov 12, 2016
@wearhere see jsdelivr/jsdelivr#13078 which has been open for over 3 months now. Again, there's nothing more we can do here on our side since we don't control JSDelivr.
wearhere commentedon Nov 13, 2016
Oh I know, I wasn't complaining. Thanks again for your research here. I was just explaining why someone might choose to use JSDelivr vs. the official CDN, I agree it would be nice to follow the official recommendation.
chore: Suggest putting `SENTRY_AUTH_TOKEN`, `SENTRY_ORG` and `SENTRY_…