search

getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.78k stars 1.53k forks source link

User mandatory in DSN #184

Closed devel-pa closed 8 years ago

devel-pa commented 10 years ago

v1.1.8 works: 'https://zuzu@localhost/LoggerSite/logging/0' doesn't: 'https://localhost/LoggerSite/logging/0' and I need the second. Thank you

mattrobenolt commented 10 years ago

How are you using Sentry without the public key?

devel-pa commented 10 years ago

I am not using Sentry, only RavenJS.

mattrobenolt commented 10 years ago

Oh, interesting.

So coming up soon, we're going to be supporting pluggable transports, and I think this would just be resolved by your own transport. Ideally, raven-js would be geared for working with Sentry out of the box then if you want to do something different, your transport would have that flexibility.

See: #183

devel-pa commented 10 years ago

No, is nothing very complex, HTTP/HTTPS. I need a regular log of the JS errors from the browser in a text file, near the rest of the server error.

mattrobenolt commented 10 years ago

Right, I understand. My point is that I don't want it to work like that out of the box because that will affect all Sentry users.

By having a separate transport, you'd be able to plug in one that was dumber than ours that didn't need to send along the auth stuff, for example.

mattrobenolt commented 8 years ago

This was resolved with https://github.com/getsentry/raven-js/pull/361