Skip to content
This repository was archived by the owner on Sep 7, 2019. It is now read-only.

Conversation

IngwiePhoenix
Copy link
Contributor

Basically small fixes, but a setting that needs improvement. I only included it as a "poke the dev"-thing ;). I used this sort of modification to track down the GET issue. I then read on the RFC entry for the WebSocket standarts and figured it expects standart HTTP headers. Well, I know that GET expects a path, not an entire URL. I yet have to figure out how to include a QueryString. But, I think one just appends it to the path... o.o" I have to try and see. I am going to investigate here and use curl -v to do so - it outputs headers too. ^^

@Devristo Devristo merged commit 604e1ae into Devristo:master Oct 1, 2013
@Devristo
Copy link
Owner

Devristo commented Oct 1, 2013

We worked paralel on the queryString support, but was able to merge the file. My test script still works, what about yours?

@IngwiePhoenix
Copy link
Contributor Author

Works still of course. :)
I am now implementing phpws into a project called appjs-deskshell, which enables desktop development with html, css and javascript. previously we only had nodejs as a backend - now however, we can have PHP too - thanks to this neat piece of code. :)
Am 01.10.2013 um 19:07 schrieb Devristo notifications@github.com:

We worked paralel on the queryString support, but was able to merge the file. My test script still works, what about yours?


Reply to this email directly or view it on GitHub.

@Devristo
Copy link
Owner

Devristo commented Oct 1, 2013

Sounds interesting, watching your branch on appjs-deskshell :)

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 this pull request may close these issues.

2 participants