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

Image Warnings... #3

Closed
bradschafer opened this issue Sep 2, 2014 · 12 comments
Closed

Image Warnings... #3

bradschafer opened this issue Sep 2, 2014 · 12 comments

Comments

@bradschafer
Copy link

@bradschafer bradschafer commented Sep 2, 2014

I am getting a lot of image warnings in Chrome...

Resource interpreted as Image but transferred with MIME type text/html: "http://logs-01.loggly.com/inputs/d62c2db7-48fa-4105-a0ca.....

It seems we're requesting an image but getting back text/html...

@hyunlim
Copy link
Contributor

@hyunlim hyunlim commented Sep 2, 2014

@bradschafer are you still having this issue? The tracker uses tracking gif and the response MIME type should be image/gif.

@bradschafer
Copy link
Author

@bradschafer bradschafer commented Sep 3, 2014

yup... I replaced actual key and params w/ xxxx for security... but it seems we're still getting text/html header response type.

REQUEST HEADERS
Remote Address:54.236.68.122:80
Request URL:http://logs-01.loggly.com/inputs/xxxxxxxxxxxxxxxxxx/tag/localhost,Console,ExtJs4,casino_2/.gif?PLAINTEXT=xxxxxxxxxxxxxxxxxx
Request Method:GET
Status Code:200 OK
Request Headersview source
Accept:image/webp,/;q=0.8
Accept-Encoding:gzip,deflate,sdch

RESPONSE HEADERS
Connection:keep-alive
Content-Length:19
Content-Type:text/html
Date:Wed, 03 Sep 2014 16:02:28 GMT
Server:nginx/1.1.19

@hyunlim
Copy link
Contributor

@hyunlim hyunlim commented Sep 3, 2014

@bradschafer Ah, I see what's going on. Unfortunately, we don't support tags for tracking pixel url endpoints right now. The current tracking pixel format we support is:

https://logs-01.loggly.com/inputs/.gif?PLAINTEXT=

It seems that the url in the tracker utility lib was modified to a different data collector url with tags in there. To get around the lack of tags support in tracking pixel GET, you can send tags as a separate field as part of the data. When we implement proper tagging at the endpoint, this library will be updated for tags as well then.

@bradschafer
Copy link
Author

@bradschafer bradschafer commented Sep 3, 2014

Yeah.. I think I rewrote that trying to segment the data sources.. since TAGS not data can be separated.
Likely we'll need to wait until the endpoint is re-written.. as the client won't goto production with errors and will need to segment the data.

@bradschafer
Copy link
Author

@bradschafer bradschafer commented Sep 4, 2014

For what it's worth.. other than the mime/type error.. the logging of the tags to that endpoint actually works..

@hyunlim
Copy link
Contributor

@hyunlim hyunlim commented Sep 5, 2014

Oh nice! For now, I'll keep this issue open, and update it when we get around to changing the endpoint. This isn't the first time proper tagging support has been requested. Thanks!

@kellycampbell
Copy link

@kellycampbell kellycampbell commented Mar 2, 2015

I get these warnings even if I don't set the tag. It uses the default tag of 'jslogger'.

vhalbwachs pushed a commit that referenced this issue Mar 12, 2015
@davidcai
Copy link

@davidcai davidcai commented Apr 28, 2015

I changed the URL to https://logs-01.loggly.com/inputs/token/.gif?PLAINTEXT=..., but I still get the warnings.

@varshneyjayant
Copy link
Contributor

@varshneyjayant varshneyjayant commented Apr 28, 2015

@kellycampbell it looks like that you are using older version of the script as current script does not use any tag

@varshneyjayant
Copy link
Contributor

@varshneyjayant varshneyjayant commented Apr 28, 2015

@davidcai I have tested with the latest script. It does not show any image warnings.

@varshneyjayant
Copy link
Contributor

@varshneyjayant varshneyjayant commented Jul 23, 2015

Try #25

@varshneyjayant
Copy link
Contributor

@varshneyjayant varshneyjayant commented Jul 24, 2015

@bradschafer Seems this to be fixed in the latest update.

@mostlyjason mostlyjason closed this Aug 4, 2015
Shwetajain148 pushed a commit to Shwetajain148/loggly-jslogger that referenced this issue Oct 23, 2017
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
6 participants
You can’t perform that action at this time.