-
Notifications
You must be signed in to change notification settings - Fork 158
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
"safety scan" requires an account and authentication #663
Comments
Hi @andy-maier, thank you for opening this issue! We appreciate your effort in reporting this. Our team will review it and get back to you soon. Note: If this is a serious security issue that could impact the security of Safety CLI users, please email [email protected] immediately. Thank you for contributing to Safety CLI! |
+1, we used |
@andy-maier and @j-adamczyk we appreciate your feedback on this and understand your concerns. I'm working through some ideas on how we can best address this, and would really value being able to discuss this with you over a call. If you're willing to do so, please email me on [email protected]. |
+2
Thanks for pointing me to |
Checklist
Safety version
3.2.14
Python version
3.12.7
Operating System
macOS 14.7.2
Describe the problem you'd like to have solved
It seems that the new "safety scan" command requires to create an account and particularly to log in to the account when running it.
I find this unacceptable for a tool that claims to be free for the open source community.
In addition, it is not clear what data is sent by the "safety scan" command to the safety site through the account.
Third, this approach can create issues when people move on from an open source project but own a personal safety account that is used for that project, and forgotten to be transferred.
Describe the ideal solution
"safety scan" does not require an account.
Alternatives and current workarounds
Workaround for us is to stick with the "safety check" command.
Additional context
No response
What I Did
The text was updated successfully, but these errors were encountered: