Re: [PEPr] Comment on HTTP::HTTP_SessionServer

Hi, Alan Knowles wrote:

as mike commented - implementing it as a session handler seems simpler on the client side. Done.

* @include (it's better to leave out the @ - if it fails, something is seriously wrong, and realistically it's probably a coding error, rather than a runtime error. Has been changed.

- I guess the define for path is a short term dev thing.. Has been removed.

* this define's a little absurd.. - you really have to be short of programming knowledge not to know what r/w/rw mean..:) define('HTTP_SESSIONSERVER_STORAGE_MODE_READWRITE', 'rw'); I'd like to keep it this way to be more flexible in the future.

the client's a bit repetative: - 60% just relays into sendCommand.. - perhaps you dont really need the individual commands..? IMHO it's easier to understand this way. The client is just an example of how to use it. You'll probably integrate it tighter in your session management, when using the SessionServer.

Thanks for your help, Stephan