Skip to content

Conversation

@tisonkun
Copy link
Member

@tisonkun tisonkun commented Aug 5, 2022

Fixes #16919

  • doc

Signed-off-by: tison <wander4096@gmail.com>
@tisonkun tisonkun marked this pull request as ready for review August 5, 2022 23:28
@github-actions github-actions bot added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Aug 5, 2022
@tisonkun
Copy link
Member Author

tisonkun commented Aug 5, 2022

cc @hpvd @sijie @merlimat @eolivelli

@tisonkun tisonkun requested a review from Anonymitaet August 8, 2022 13:14
@Anonymitaet Anonymitaet merged commit b1ad198 into apache:master Aug 9, 2022
@tisonkun tisonkun deleted the security-policy branch August 9, 2022 03:25
@lhotari
Copy link
Member

lhotari commented Aug 9, 2022

I hope that the information about the vulnerability handling process could also be added to SECURITY.md . Duplication of information is justified in this case. Hiding relevant information behind yet another click could result in information being missed.

I'd suggest that we also add these sentences to SECURITY.md:

The Pulsar community follows the ASF vulnerability handling process.

To report a new vulnerability you have discovered please follow the ASF vulnerability reporting process.

@lhotari
Copy link
Member

lhotari commented Aug 9, 2022

Oh, I see that @tisonkun already suggested inlining the information in #16919 (comment) .

@tisonkun
Copy link
Member Author

tisonkun commented Aug 9, 2022

@lhotari I'm OK to duplicate the information but the current state is acceptable for me and I think it resolves the original issue.

If you'd like to duplicate the information, you can submit a patch :)

@lhotari
Copy link
Member

lhotari commented Aug 10, 2022

@lhotari I'm OK to duplicate the information but the current state is acceptable for me and I think it resolves the original issue.

If you'd like to duplicate the information, you can submit a patch :)

@tisonkun I made #17039 as a follow-up

Technoboy- pushed a commit to merlimat/pulsar that referenced this pull request Aug 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Security] [Doc] no advice on how to report vulnerabilities

4 participants