ucla-cs130 / team12 Goto Github PK

Hi Team12,

We found two issues while trying to implement the proxy handler.

1. In Server::session, if the request uri is /foo/bar/abc/def, you currently only look for /foo (which is req->uriHead()) in the uri2handler map. This is incorrect, because a handler's path in the config may be /foo/bar. Therefore, you must check for all possible combinations in uri2handler and pick the handler with the longest matching uri prefix:

   • /foo/bar/abc/def
   • /foo/bar/abc
   • /foo/bar
   • /foo

2. The proxy handler will only work correctly if its path is set to / in the config. Currently, if the request uri is /, it will be changed to /index by Request::uriHead which is incorrect. It should be left as /.

Once these two issues are fixed, the proxy handler will work correctly.

Thanks!
spaceteam