Skip to content
This repository has been archived by the owner on Jul 19, 2019. It is now read-only.

Tutorial uses POST with SimpleHTTPServer #1

Closed
simonwhitaker opened this issue Aug 5, 2014 · 13 comments
Closed

Tutorial uses POST with SimpleHTTPServer #1

simonwhitaker opened this issue Aug 5, 2014 · 13 comments

Comments

@simonwhitaker
Copy link

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.

@chenglou
Copy link
Contributor

chenglou commented Aug 5, 2014

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

@haffla
Copy link

haffla commented Nov 15, 2014

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!

@trkrameshkumar
Copy link

is this still need to be done ?

@zpao
Copy link
Member

zpao commented Nov 26, 2014

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

@dmonopoly
Copy link

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

@ilmoralito
Copy link

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

@renaldow
Copy link

renaldow commented Jan 5, 2015

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

@AndrewSouthpaw
Copy link

+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.

@tbredin
Copy link

tbredin commented Jan 10, 2015

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

@chenglou
Copy link
Contributor

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

@zpao
Copy link
Member

zpao commented Jan 11, 2015

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

@zpao
Copy link
Member

zpao commented Jan 16, 2015

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

@zpao zpao closed this as completed Jan 16, 2015
@hellyeah
Copy link

Someone should just link to this issue in the tutorial.

shenxl pushed a commit to shenxl/react-tutorial that referenced this issue Sep 15, 2015
Fix typo and minor wording
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.