video14

[mcharlie9]

At 0:15, you mentioned that when going to localhost:3000, everything will be working.

However, after starting the server, the routes don't seem to be working. For instance, GET /recipes returns 404 "Not Found", instead of "list recipes". Would you mind checking if you are also seeing the same thing? Thanks.

[jacekschae]

Yeah. Planning to dig into this in the next video. I think the main problem is the config in the router. If you update as following, it should work. Note the :host.

(def routes
  (route/expand-routes
    #{{:app-name :cheffy :schema :http :host "localhost"}
      ["/recipes" :get list-recipes :route-name :list-recipes]
      ["/recipes" :post upsert-recipe :route-name :create-recipe]
      ["/recipes/:recipe-id" :put upsert-recipe :route-name :update-recipe]}))

[mcharlie9]

Just looking at the codes, and noticed the (def routes ...) is still showing :schema instead of the correct :scheme. Can cross the bridge when we come to it, but I do wonder when this keyword would be used and whether it would have any "side effect" as the course continues.

[jacekschae]

I should have fixed it within 21. Shame that I missed this one. I'll circle back to this one.

[mcharlie9]

Just spent quite some time on video21 since I cannot get the pedestal test to return 200. Finally, I realize that by changing :schema to the correct :scheme, the pedestal test would return 500 instead. Seems like we have already come to the bridge regarding this typo. It is interesting that the codes with the typo work because the typo KV pair is ignored and the default value is used. I wonder why the code is acting up...

Steps to reproduce:

1. Change :schema to :scheme
2. (restart-dev)
3. (pt/response-for ...)

Would you mind taking a look to see if you are also getting 500 error with the above? It would be nice to figure out why this is the case.

[jacekschae]

Same things. If you fully namespace :scheme to ::http/scheme the int's all fine. I'll probably take a second take on the 21.

[mcharlie9]

Just figured it out too. Thanks for letting me know. Yes, a second take might be simpler. Will send through a few more feedbacks regarding video21 on a new discussion thread.