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

Take care of files without extension #566

Closed
wants to merge 1 commit into from
Closed

Conversation

ReiatsuSan
Copy link

If not set as a default it could crash in an internal server error

@GrahamCampbell
Copy link
Member

Thanks, but this is not true. If there's no extension, both public methods in this class return null.

@GrahamCampbell
Copy link
Member

If your code is assuming non-null, then the bug is there. If there is code within Guzzle that makes that bad assumption, please point me in that direction.

@ReiatsuSan
Copy link
Author

ReiatsuSan commented Jul 19, 2023

Hi @GrahamCampbell, it's effectively an issue within psr7 in the MultipartStream class inside createElement method (https://github.com/guzzle/psr7/blob/815698d9f11c908bc59471d11f642264b533346a/src/MultipartStream.php#L141C1-L141C1).
There is an assumption a content-type is returned no matter what...

@GrahamCampbell
Copy link
Member

Thanks. Do you mind to open an issue?

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.

2 participants