-
Notifications
You must be signed in to change notification settings - Fork 33
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
Bower support + Firefox NX source download bug #13
Comments
|
GA?
Spyware detected? nx-beta.2.0.0.js what do these questions have to do with knockout whatsoever? |
@hardhub GA is Google Analytics. Whether you think that's spyware because it's Google is up to you, lol. |
@NetOperatorWibby LOL? LOL or lol is acronym for laugh out loud... why are you laughing without a reason? Are you idiot? It was not mentioned as real malware.. But behavior of the framework is the same as for spyware. This framework tracks users of all apps loading mentioned js-file with GA... But framework is NOT intended to do such things... It is UI framework.. Why should I inject framework GA in my app? It is even never mentioned in any documents... Privacy of users of my app are not important? Then I say this library goodbye. It is not the way for OSS. Even M$ claims about opt out telemetry in .net core tools... bad BUT NOT HIDDEN and with option to DISABLE. So yes it looks really like a Trojan horse! |
Guys it would be great if we all could behave with maturity and keep away from using foul languages. If GA seems a big issue then please raise an issue(which is already done here) and let the library maintainers look into it. @solkimicreb can you please look into it? |
Hi @hardhub ! The framework is 12kb minzipped if everything is included (the next version will be minified by default). Unminified and unzipped it is 92kb. It definitely has no GA inside it. What you opened is the source code of the nx home page (yes, the home page has GA tracking). The link that you provided is a download link, if you open it in the browser it points to the source of the nx home page. If you open it by clicking the actual download link here, it sets the correct headers and points to the nx source code. This is done to work around a problem in Safari. I hope it clarified things for you. Bower is not supported yet. I will try to add it to the next release (scheduled in approx a week). Edit: If you are interested in the source code check the repos in this GitHub organization |
Hi @solkimicreb Now all much more clear. But I think it does not work as you described it. At least it happens in firefox v51. |
You are right, it does happen in Firefox. I checked some other browsers and it works correctly in Chrome, Safari and Opera. I think that the download hack, that I used for Safari broke it. I will look into it tomorrow. Thanks for the bug catch (: |
@hardhub I wasn't laughing at you, it was more for me as I often view Google as malware. Like @vivekimsit mentioned, your language was unnecessary and your reaction was overboard. |
@NetOperatorWibby
All what is for you should be at you... I think reasonable...
It is not for you to decide... I do not like your reaction no less. But I do not think we need to discuss it anymore... Originally I told about technical issues.. they are now clear.. so we just need to solve it... nothing else... |
I will keep this issue open until the firefox download bug is fixed and bower support is added. I changed the issue title a bit, @hardhub I hope you don't mind. |
Firefox issue is fixed, the next release will take some more time though (It got bigger then I originally planned). I will keep the issue open till bower support lands. |
I need to clarify some questions:
The text was updated successfully, but these errors were encountered: