Added Cookie Feature. Now you can use Cookies with this restClient.#16
Added Cookie Feature. Now you can use Cookies with this restClient.#16chfo wants to merge 1 commit intomrtazz:masterfrom
Conversation
|
This is awesome! Thanks for the contribution! Unfortunately it seems that this patch breaks the tests. Could you look into that? It would also be great to have unit tests for this new functionality. I was looking into changing over the tests to use http://httpbin.org anyways, so you can totally use that as a source for integration data. |
|
Hi. I will try to fix this and give you some unit tests. |
|
Hi, After that i tested it in my terminal: Still Code 301. //The master branch test is still failing too. Best regards |
|
the current unit test suite in master now also supports JSON parsing (#25) |
|
@chfo do you have time to make those adaptions so this can be merged? |
|
This won't apply to master anymore. I'd still be interested in this functionality if you have the time to base it off the current design. Closing this in the meantime. |
With this simple expansion the Rest Client is able to use Cookies. It saves the cookies to a file 'cookie' and use it with every following request. This can be used for a simple authentification check, which should be part of a restfull service.