Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Do not send parameters with undefined to body #719 #722

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Do not send parameters with undefined to body #719 #722

wants to merge 1 commit into from

Conversation

youngkaneda
Copy link

@youngkaneda youngkaneda commented Feb 9, 2019

This prevents a undefined body to be passed on URL parameters

    if (value === undefined) {
        return;
    }

@youngkaneda
Copy link
Author

youngkaneda commented Mar 11, 2019

@janschoenherr could you review this?

@NarHakobyan
Copy link

NarHakobyan commented Mar 14, 2019

is this project maintaining? I think need to create new maintainable library :/

@youngkaneda
Copy link
Author

maybe not, would be nice if the project dev had not discontinued.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants