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

Browser is requesting source map, which doesn't exist on the CDN #36

Closed
johanbrook opened this issue Feb 15, 2016 · 12 comments
Closed

Browser is requesting source map, which doesn't exist on the CDN #36

johanbrook opened this issue Feb 15, 2016 · 12 comments

Comments

@johanbrook
Copy link

@johanbrook johanbrook commented Feb 15, 2016

Hi there. We're loading the Loggly JS client async from your site, like this:

load('//cloudfront.loggly.com/js/loggly.tracker.min.js', function() {
  // Init functions
});

The minified version will generate a sourcemap URL line at the bottom:

//# sourceMappingURL=loggly.tracker.min.map

which makes the browser request http://cloudfront.loggly.com/js/loggly.tracker.min.map, which doesn't exist. This produces an ugly 404 error in the JS console in browsers ...


Can you either put a source map file at http://cloudfront.loggly.com/js/loggly.tracker.min.map, or remove the source map line from the minified version of the client?

Thanks!

@fredericnowak
Copy link

@fredericnowak fredericnowak commented Feb 29, 2016

+1

@janderson-beapro
Copy link

@janderson-beapro janderson-beapro commented Mar 8, 2016

👍

2 similar comments
@nporter-pa
Copy link

@nporter-pa nporter-pa commented Mar 8, 2016

+1

@kittrCZ
Copy link

@kittrCZ kittrCZ commented Apr 14, 2016

+1

@mostlyjason
Copy link
Contributor

@mostlyjason mostlyjason commented May 11, 2016

Thanks guys we'll try to get this updated in one of our upcoming deployments

@rlancer
Copy link

@rlancer rlancer commented May 23, 2016

+1

2 similar comments
@craigfurlong
Copy link

@craigfurlong craigfurlong commented Jun 3, 2016

+1

@stefy
Copy link

@stefy stefy commented Jun 14, 2016

+1

@mostlyjason
Copy link
Contributor

@mostlyjason mostlyjason commented Aug 1, 2016

This has been fixed in #40

@mostlyjason mostlyjason closed this Aug 1, 2016
@johanbrook
Copy link
Author

@johanbrook johanbrook commented Oct 26, 2016

It appears to still persist for us. See https://cloudfront.loggly.com/js/loggly.tracker.min.map : it's a 404. The JS file in question, https://cloudfront.loggly.com/js/loggly.tracker.min.js, specifies

//# sourceMappingURL=loggly.tracker.min.map

at the bottom.

@mostlyjason
Copy link
Contributor

@mostlyjason mostlyjason commented Nov 22, 2016

@johanbrook you should include the latest version number in your URL. We have kept this old one for legacy reasons but we encourage people to use the version numbers going forward so that people can control when they upgrade http://cloudfront.loggly.com/js/loggly.tracker-2.1.min.js

@johanbrook
Copy link
Author

@johanbrook johanbrook commented Nov 22, 2016

Oh, good to know. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
9 participants
You can’t perform that action at this time.