Skip to content
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

Remove custom security button as GitHub added own #1157

Merged
merged 1 commit into from Aug 28, 2023
Merged

Conversation

rugk
Copy link
Member

@rugk rugk commented Aug 28, 2023

Apparently GitHub now adds a security policy button by default (this is new, is not it?)? Also they have a policy report form behind that button. So reports can apparently now be made online at GitHub? (IMHO that is fine, just need to be aware of that)
grafik

As such, IMHO two buttons would be confusing, so let's remove our custom one here?

Apparently GitHub now adds a security policy button by default (this is new, is not it?)?
Also they have a policy report form behind that button. So reports can apparently now be made online at GitHub? (IMHO that is fine, just need to be aware of that)

As such, IMHO two buttons would be confusing, so let's remove our custom one here?
@rugk
Copy link
Member Author

rugk commented Aug 28, 2023

As you can see in cryptpad/cryptpad#1220 I have the same issue in other repos, so apparently, also if I have no special access to the repo as here…

Also maybe we should link that new form in our security policy and security.txt then?

Copy link
Contributor

@elrido elrido left a comment

Choose a reason for hiding this comment

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

@elrido elrido merged commit e6f2ad0 into master Aug 28, 2023
15 checks passed
@elrido elrido deleted the securitybutton branch August 28, 2023 18:42
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.

None yet

2 participants