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

util/mime: pass invalidSubtypeIndex instead of trimmedSubtype to error #49711

Closed
wants to merge 1 commit into from
Closed

util/mime: pass invalidSubtypeIndex instead of trimmedSubtype to error #49711

wants to merge 1 commit into from

Conversation

gaurishhs
Copy link
Contributor

The error ERR_INVALID_MIME_SYNTAX expects the invalid index which wasn't being passed.

@nodejs-github-bot nodejs-github-bot added the needs-ci PRs that need a full CI run. label Sep 19, 2023
@gaurishhs gaurishhs changed the title fix: pass invalidSubtypeIndex instead of trimmedSubtype to error mime: pass invalidSubtypeIndex instead of trimmedSubtype to error Sep 19, 2023
@gaurishhs gaurishhs changed the title mime: pass invalidSubtypeIndex instead of trimmedSubtype to error util/mime: pass invalidSubtypeIndex instead of trimmedSubtype to error Sep 19, 2023
@nodejs-github-bot

This comment was marked as outdated.

@gaurishhs
Copy link
Contributor Author

hi, any updates?

@nodejs-github-bot
Copy link
Collaborator

@deokjinkim
Copy link
Contributor

@gaurishhs Could you modify commit title according to guide(https://github.com/nodejs/node/blob/main/doc/contributing/pull-requests.md#commit-message-guidelines)?
Commit title should be started with submodule name such as util.

@gaurishhs
Copy link
Contributor Author

@deokjinkim @jasnell I was having conflicts on my local git while rebasing. I have opened a new PR to make things good. Please look into #51264

I'm sorry for the mess i created

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-ci PRs that need a full CI run.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants