reactjs/react-tutorial

Tutorial uses POST with SimpleHTTPServer

simonwhitaker opened this issue · 13 comments

The tutorial suggests using python -m SimpleHTTPServer to serve content, but subsequently introduces a mechanism for submitting comments via a POST request, which SimpleHTTPServer doesn't support. It appears that at this point the tutorial assumes you're using the sample node.js server packaged with the tutorial's GitHub repo but that isn't explained.

Nice catch. Wanna send a PR to the react repo?

I was going to open an issue about that and then saw that simonwhitaker already did that.

When I did the tutorial I thought I did something wrong and spend an hour of googling the issue:
Using python simple http server the error message is 501" Unsupported method ('POST')"
Using Node's http-server or proper Apache server the error is a 404.. comment.json not found

I then went to this repo and took your server.js and it worked. So this should really be updated to prevent frustration. Thank you anyway!

is this still need to be done ?

zpao commented

We still need to update the tutorial on the site but the project here no longer uses simple server.

I encountered this same issue - it would be good to update the tutorial =].

501 error when try to post comments using python -m simpleHTTPServer as a server, i can see node, python, ruby servers in repo but i think this should be pointed in documentation

I ran into the same 501 error using the POST request in the simpleHTTPServer method. Thanks for creating a PR simonwhitaker.

+1 to Simon's PR. Looks like everything is there. I actually wrote out my own solution before learning (through Simon) that a server solution already exists in the repo.

This caused me about 2 hours of despair before finding this issue, so sure I'd done everything right following the tutorial perfectly.. :(

+1 Simon's PR looks pretty worthwhile

@zpao kinda lost track here, what's happening?

zpao commented

facebook/react#2824, which is similar to @simonwhitaker's PR. Or we could go back to his and tweak it some.

zpao commented

Updating the website shortly with newer docs so this is fine to close now.

Someone should just link to this issue in the tutorial.