Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove secret_key from required keys #645

Merged
merged 1 commit into from
Aug 16, 2018
Merged

Conversation

spras
Copy link
Contributor

@spras spras commented Aug 16, 2018

#644

This PR fix the issue :

$ bin/sentry test https://ffd1aa57b86643148bb64220496ba6a0@sentry.io/1263429 -v
Client configuration:
-> server: https://sentry.io/api/1263429/store/
-> project: 1263429
-> public_key: ffd1aa57b86643148bb64220496ba6a0

Sending a test event:
-> event ID: 34227d39711b438bbf895e092b1eec82

Done!%

@stayallive
Copy link
Collaborator

Hmm that is odd, that should have been removed a long time ago :)

Anyhow, thanks for making the PR! 👍

@stayallive stayallive merged commit 4df912c into getsentry:master Aug 16, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants