eventsource icon indicating copy to clipboard operation
eventsource copied to clipboard

Who to contact for security issues

Open JamieSlome opened this issue 3 years ago β€’ 9 comments

Hey there!

I belong to an open source security research community, and a member (@sampaguitas) has found an issue, but doesn’t know the best way to disclose it.

If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.

Thank you for your consideration, and I look forward to hearing from you!

(cc @huntr-helper)

JamieSlome avatar Feb 07 '22 21:02 JamieSlome

Just for reference, the report can be found here: https://huntr.dev/bounties/dc9e467f-be5d-4945-867d-1044d27e9b8e/

It is private and only accessible to maintainers with repository write permissions πŸ‘

JamieSlome avatar Apr 30 '22 15:04 JamieSlome

(cc) @rexxars πŸ‘†

JamieSlome avatar May 01 '22 11:05 JamieSlome

Hi, sorry for hijacking this issue but could you please take a look at approving v1.1.1 for fixing the above mentioned issue too? I am using a tool called snyk and the CVE seems to now acknowledge version 1.1.1 to contain the fix just yet.

ghost avatar May 20 '22 12:05 ghost

could you please take a look at approving v1.1.1 for fixing the above mentioned issue too?

Do you know how I could go about doing so?

rexxars avatar May 20 '22 13:05 rexxars

@rexxars - I might be able to help here. If you can confirm that v1.1.1 is not affected by this, I can update the CVE πŸ‘

See: https://github.com/418sec/cvelist/blob/master/2022/1xxx/CVE-2022-1650.json

JamieSlome avatar May 20 '22 14:05 JamieSlome

could you please take a look at approving v1.1.1 for fixing the above mentioned issue too?

Do you know how I could go about doing so?

Sorry, but I'm not familiar with this process at all :( Jamie seems to be able to assist with it though πŸ’―

ghost avatar May 20 '22 14:05 ghost

Hi Jamie,

Just to make sure: we are discussing about the fix bounty here and not the vulnerability finding.

Both of these versions where not out when I wrote the report 3 months ago, and at that time the latest version was vulnerable to what I have described in the POC.

With best regards,

Timothee

Get BlueMail for Androidhttps://bluemail.me On May 20, 2022, at 18:32, jhspinpanel @.@.>> wrote:

could you please take a look at approving v1.1.1 for fixing the above mentioned issue too?

Do you know how I could go about doing so?

Sorry, but I'm not familiar with this process at all :( Jamie seems to be able to assist with it though πŸ’―

β€” Reply to this email directly, view it on GitHubhttps://github.com/EventSource/eventsource/issues/244#issuecomment-1132968198, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJTI33JGR5GU2DPV2CGWZW3VK6PALANCNFSM5NYTFOAA. You are receiving this because you were mentioned.Message ID: @.***>

Sampaguitas avatar May 20 '22 19:05 Sampaguitas

@rexxars - I might be able to help here. If you can confirm that v1.1.1 is not affected by this, I can update the CVE πŸ‘

See: https://github.com/418sec/cvelist/blob/master/2022/1xxx/CVE-2022-1650.json

To summarize:

  • v1.1.0 and below are affected by this issue - the issue is fixed in v1.1.1.
  • v2.0.0 and v2.0.1 are affected by this issue - the issue is fixed in v2.0.2.

This ensures that people do not have to upgrade to v2 if they are stuck on the v1 range, due to needing support for node 12 or below, or similar.

Would be great if you could update the CVE.

rexxars avatar May 23 '22 17:05 rexxars

@rexxars - sorted:

https://github.com/CVEProject/cvelist/pull/5853/files

JamieSlome avatar May 24 '22 10:05 JamieSlome