python-multipart: initial integration
Hi, python-multipart is a streaming multipart parser for Python. It has 2 million+ monthly downloads as per pypistats, and is being used by projects like fastapi, starlette.
Hi @jonathanmetzman please review this pr.
Hi @Navidem please review this pr :)
Has upstream agreed to this?
Has upstream agreed to this?
@jonathanmetzman There's no response yet.
Has upstream agreed to this?
@jonathanmetzman There's no response yet.
We don't accept projects unless upstream agrees
Has upstream agreed to this?
@jonathanmetzman There's no response yet.
We don't accept projects unless upstream agrees
@jonathanmetzman thanks for the review, can you please clear doubt of mine. I can see pr https://github.com/google/oss-fuzz/pull/8347 with no response being merged. Whats the difference here? Just curious thanks.
cc @oliverchang
Has upstream agreed to this?
@jonathanmetzman There's no response yet.
We don't accept projects unless upstream agrees
@jonathanmetzman thanks for the review, can you please clear doubt of mine. I can see pr #8347 with no response being merged. Whats the difference here? Just curious thanks.
These are our contractors and we treat these projects differently.
These are our contractors and we treat these projects differently.
@jonathanmetzman a previous pr: https://github.com/google/oss-fuzz/pull/8279 of mine was merged same way, that's why it was creating confusion for me.
I probably shouldn't have merged it because there's no one upstream to accept bug reports, but there's a linked issue where upstream agreed to the integration. Here there is nothing
I probably shouldn't have merged it because there's no one upstream to accept bug reports, but there's a linked issue where upstream agreed to the integration. Here there is nothing
Yes, that was because maintainer agreed a day or two later. and thanks for clearing my doubts I'll keep this in mind for my contributions :)
I probably shouldn't have merged it because there's no one upstream to accept bug reports, but there's a linked issue where upstream agreed to the integration. Here there is nothing
Yes, that was because maintainer agreed a day or two later. and thanks for clearing my doubts I'll keep this in mind for my contributions :)
Yeah youre right, their agreement was later. I shouldn't have merged it but it ended up being OK.