jkrall / analytical

Gem for managing multiple analytics services in your rails app.
MIT License
380 stars 92 forks source link

Set support for MixPanel #9

Closed kwerle closed 13 years ago

kwerle commented 13 years ago

No spec test, but it sure seems to work.

jkrall commented 13 years ago

Looks OK... not sure about the forcing of https, but I merged it in anyway, since I don't use MixPanel (and therefore can't support it myself). If you say it works, that's good enough for me.

kwerle commented 13 years ago

On Mar 14, 2011, at 10:46 PM, jkrall wrote:

Looks OK... not sure about the forcing of https, but I merged it in anyway, since I don't use MixPanel (and therefore can't support it myself). If you say it works, that's good enough for me.

Ugh- totally the wrong thing to do. Well, not totally. It should use protocol - but I had trouble getting that working for some reason and https is 'always safe', so I punted. The issue with getting protocol to work might have just been that the gem was failing to refresh - I certainly did have issues.

Anyway, https is a better default than http, at least.

Kurt

http://www.CircleW.org/kurt/