Skip to content

Conversation

@cored0wn
Copy link

Let the services.yml load automatically instead of configuring them in the project specific services.yml

See https://symfony.com/doc/current/bundles/extension.html

so the services.yml file would be loaded automatically
Removed unnecessary installation notice
@cored0wn
Copy link
Author

Updated README.md so the services wouldn't be defined twice 😉

Christopher Wittor added 4 commits August 7, 2018 11:17
 - return correct content-id
 - some PHP_EOL added
 - added handling of multipart/form-data request
@cored0wn
Copy link
Author

cored0wn commented Aug 7, 2018

Added support for multipart/form-data subrequest.

You can now start request like this

POST /batch HTTP/1.1
Host: example.com
content-type: multipart/batch; type="application/http;version=1.1"; boundary=batch

--batch
Content-Type: application/http;version=1.1
Content-Transfer-Encoding: binary
Content-ID: <abcd-1234@example.org>

POST /oauth2/token HTTP/1.1
Host: example.com
content-type: multipart/form-data; boundary=--form1

----form1
Content-Disposition: form-data; name="grant_type"

refresh_token
----form1
Content-Disposition: form-data; name="refresh_token"

44b66d4b02b5c47b5c003e2d5bd443c83157bf445f4f151dbc6979d69e32e2ff9e4e8eefd176ce10d49aac
----form1--
--batch
Content-Type: application/http;version=1.1
Content-Transfer-Encoding: binary
Content-ID: <zyxw-9876@example.org>

POST /oauth2/introspect HTTP/1.1
Host: example.com
content-type: multipart/form-data; boundary=--form2

----form2
Content-Disposition: form-data; name="token"

fe73b260e105b6ce99ec0638bc2f3e76e5241faff8c5588588c27f8944aaa832128233992b4b306dcb3146
----form2--
--batch--

@cored0wn
Copy link
Author

I have added some features, e.g. the possibility to limit the number of calls via the configuration.
I also changed the HttpHeaderException to the Symfony BadRequestHttpException, so you don't need ext-http anymore.

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