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

Browsing node paths containing a period (".") break with PHP built-in web server #20

Closed
bvleur opened this issue Mar 27, 2014 · 5 comments

Comments

@bvleur
Copy link
Contributor

bvleur commented Mar 27, 2014

The README says it's better to use the PHP built-in webserver but API calls with containing periods confuse that server to go look for a specific file from the docroot instead of using index.php.

I see the following possible resolutions:

  1. Change the API url prefix from /api to /index.php/api in api-foundation.js line 5
  2. Don't use "docroot" functionality (the -t parameter) of the PHP built-in web server but supply a routing.php file like suggested here.
  3. Warn against this "known issue" when advising the PHP built-in webserver (or stop suggesting it at all).
  4. URL encode periods (don't know if works somehow).
@bvleur
Copy link
Contributor Author

bvleur commented Mar 27, 2014

An alternative to resolution 2 is in the Silex docs already:
http://silex.sensiolabs.org/doc/web_servers.html#php-5-4

@RobinBressan
Copy link
Contributor

Thanks for this report!
I think it is better to warn against this "known issue" when advising the PHP built-in webserver.
Could you make a quick PR to edit the README?

bvleur added a commit to bvleur/phpcr-browser that referenced this issue Mar 30, 2014
RobinBressan added a commit that referenced this issue Mar 30, 2014
Include known issue with PHP webserver and periods in URLs in the README (fixes #20)
@fzaninotto
Copy link
Member

If the Silex configuration makes the trick, why don't you implement it? IMO, it's better to provide a plug and play solution, and the PHP built-in server allows for that.

@bvleur
Copy link
Contributor Author

bvleur commented Mar 31, 2014

@fzaninotto It was a solution I preferred above just adding a warning. But as an outsider I feel owners should decide about their repository.

@fzaninotto
Copy link
Member

@RobinBressan I think you should implement the Silex trick to make php server a first-class setup option.

@RobinBressan RobinBressan mentioned this issue Apr 27, 2014
2 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants