-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
secret_key not included #263
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
Comments
raven-js should not use your secret key. That's extremely insecure. The issue is that your request isn't attaching an Origin or Referer header, so its' falling back to the secret key auth. This was not very pretty in 6.4.4, but in master, it should more explicitly tell you that. Feel free to upgrade your Sentry to master. It's what we run on app.getsentry.com. I can help you try and figure out why your Origin and Referer header isn't getting attached to your request though. |
Hi Matt, I upgraded to the master branch, yes the response is a lot more helpful On Mon, Sep 29, 2014 at 7:27 AM, Matt Robenolt [email protected]
|
Usually the first guess is that you're making a request across protocols. So maybe your page is https and your DSN is using http. Or vice-versa. Can you show me the request that's being attempted? |
Weird.. so it sent your cookies not not a Referer. It seems that I can't access this louiep.restlessthinker.com to test personally. Is there a way that I can? |
Not at the moment, that domain is on aws only accessible from an internal On Tue, Sep 30, 2014 at 12:14 PM, Matt Robenolt [email protected]
|
I was having the same issue here: #29 (comment)
The problem is that ravenjs isn't sending secret_key that's being used here:
https://github.com/getsentry/sentry/blob/master/src/sentry/web/api.py#L236
If you include public:secret in the DSN, raven now throws an error that doesn't allow it, but it's not added here https://github.com/getsentry/raven-js/blob/master/src/raven.js#L453.
I did hardcode and appended "sentry_secret=" and it finally worked. I can fix it and submit a pull request but how would you like that DSN check handled? Thanks and I look forward to your reply!
Raven-js 1.1.16 and Sentry 6.4.4
The text was updated successfully, but these errors were encountered: