[go: nahoru, domu]

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

add form example #765

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

add form example #765

wants to merge 1 commit into from

Conversation

smallfish
Copy link
@smallfish smallfish commented Dec 19, 2020

for #761

@smallfish smallfish changed the title add form example #761 add form example Dec 19, 2020
@masklinn
Copy link
Contributor

FWIW should probably clearly specify that the form is urlencoded (maybe provide HTTP endpoints to POST there as well to avoid the need for comments?): even though that's specified by the documentation for warp::filters::body::form we can't assume the reader will go check that out, especially since it takes no parameters or options. Otherwise there's 100% chance people will be confused when their multipart/form-data doesn't work properly.

@jxs jxs added the waiting-on-author awaiting some action (such as code changes) from the PR or issue author. label Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-on-author awaiting some action (such as code changes) from the PR or issue author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants