Skip to content

Handle form-encoded POST bodies #3

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.

Sign up for GitHub

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

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

Handle form-encoded POST bodies #3

jamietanna wants to merge 1 commit into from

Conversation

@jamietanna
Copy link
Contributor

@jamietanna jamietanna commented Aug 24, 2022

As RFC6750 allows form-encoded bodies, we should add support for this.

This adds new subtests for the form-encoded body tests, as well a
simplifying the logic for looking up the value of the token in the
request.
@jamietanna
Handle form-encoded POST bodies
4ec762d 
As RFC6750 allows form-encoded bodies, we should add support for this. This adds new subtests for the form-encoded body tests, as well a simplifying the logic for looking up the value of the token in the request.
@jamietanna
Copy link
Contributor Author

jamietanna commented Aug 24, 2022 •
edited
Loading

I'm tempted to have multipart/form-data supported too, but it's a little more awkward, so happy sticking with this for now, as it'll unblock a few apps I use

(Until this is merged, I'm pinning to my fork)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant

@jamietanna