Skip to content

docs: create Security Policy #530

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

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Conversation

diogoteles08
Copy link
Contributor

Closes #529

I've created the SECURITY.md file following a GitHub's template and considering that you'd request that users report vulnerabilities through the security advisory, which is a handy new GitHub feature, but it's still in beta and has to be manually enabled by a maintainer.

If you're interested in this feature, you can activate it following this steps:

  1. Click on this link to go to Code security & analysis section on your repo's settings
  2. Click "Enable" for "Private vulnerability reporting (Beta)"

However, if you'd rather not use this feature, you can also request users to report vulnerabilities to an email. If that's the case, let me know which email you would like to receive the reports and I can submit the change.

Additionally, feel free to edit or suggest any changes to this document, it is supposed to reflect the amount of effort the team can offer to handle vulnerabilities.

Copy link
Contributor

@droideck droideck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The doc and approach look good to me.

@mistotebe what do you think?

@mistotebe
Copy link
Contributor

LGTM!

@diogoteles08
Copy link
Contributor Author

That's great!
It seems we need to have all checks working prior to merge but one of them didn't pass. I don't think it has anything to do with my change, tho. Can you help me out here? Seems like I'm unable to re-run the failing test :/

@droideck droideck merged commit 72c1b5e into python-ldap:main Jul 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create a Security Policy
3 participants