resttesttest icon indicating copy to clipboard operation
resttesttest copied to clipboard

Added a Request Body textarea along with a corresponding Content-Type…

Open philip opened this issue 8 years ago • 8 comments

Added a Request Body textarea along with a corresponding Content-Type definition. Code taken directly from the following fork: https://github.com/taggiasco/resttesttest Patches in that fork included many other changes, so can't provide a direct link.

philip avatar Dec 20 '16 23:12 philip

This pull request also helps with Issue #7

philip avatar Dec 20 '16 23:12 philip

Hi, How do you deal with parameters ? Since you have a body, parameters are ignored for the moment. In my fork, I added a "buildArguments" which appends the parameters to the URL (only when you have a body). Not sure if it is the right way of doing this, but the request seems to be correctly handled by the test service (https://httpbin.org/put). Best regards, Matt

taggiasco avatar Dec 21 '16 07:12 taggiasco

@taggiasco Ah, an excellent point, thank you. My attempts to keep the pull request as simple/small as possible missed that fact, and my tests all used headers+body and not parameters. I'll update the pull request at some point.

philip avatar Dec 21 '16 16:12 philip

@taggiasco @jeroenooms Okay, this merge request seems fine now. I don't fully understand this topic yet as I'm still learning the ins and outs of REST APIs (and HTTP requests in general).

philip avatar Dec 21 '16 17:12 philip

Is there any plan to finally merge this and add this functionality?

bracco23 avatar Oct 19 '17 08:10 bracco23

@bracco23 This project is no longer active (no commits in 2+ years) so I doubt it.

philip avatar Oct 19 '17 16:10 philip

You know you can easily deploy your own version right? It's just static html. You can upload it to anywhere you want and it'll work :)

jeroen avatar Oct 19 '17 16:10 jeroen

@jeroen True, although it's easier to click a button (download) than worry about applying patches :) But I suspect you'll not merge this PR as to keep https://resttesttest.com/ as is, right? Understandable, of course.

philip avatar Oct 24 '17 01:10 philip