-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
First draft #1
First draft #1
Conversation
With the changes mentioned in https://github.com/potherca-contrib/solid-php/pull/1 I was able to start the Docker container. I didn't see anything on the host system: loopy:php-solid-server michiel$ curl http://localhost/
curl: (7) Failed to connect to localhost port 80: Connection refused Then I ran a shell inside the container so that I could see http://localhost/ on the inside, and this is the error I saw: loopy:php-solid-server michiel$ docker ps
CONTAINER ID IMAGE COMMAND CREATED STATUS PORTS NAMES
3c7f24138c92 php:7.1 "docker-php-entrypoi…" 3 seconds ago Up 3 seconds interesting_lehmann
loopy:php-solid-server michiel$ docker exec -it 3c7f /bin/bash
root@docker-desktop:/# curl http://localhost/
<br />
<b>Warning</b>: require(/app/web./../vendor/autoload.php): failed to open stream: No such file or directory in <b>/app/web/index.php</b> on line <b>5</b><br />
<br />
<b>Fatal error</b>: require(): Failed opening required '/app/web./../vendor/autoload.php' (include_path='.:/usr/local/lib/php') in <b>/app/web/index.php</b> on line <b>5</b><br />
root@docker-desktop:/# Should I have run EDIT: I also tried with the instructions based on the "composer" Docker image, but same result. |
Ugh. The culprit is I've also merged you fix for docker 👍 Regarding composer install... I'm not entirely shure what the best way to go is. When I have sorted out a Dockerfile for this repo, it will indeed run composer install as one of the layers. Not entirely sure how/what to do with that in the docs for other solutions. I'll think about that a bit today and update the docs. |
With the composer Docker image, it's possible to run |
Now, I was able to see 'hello world' :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! I'll add it to https://github.com/pdsinterop/test-suites/tree/master/servers
This MR adds some first-draft documentation and an empty PHP HTTP server (based on several popular PHP packages).
For all intents and purposes,this can be considered version
0.1.0
.