fix clean return path user code handle

Signed-off-by: Andy Green <andy.green@linaro.org>

add explicit error for partial send

This patch adds code to handle the situation that a prepared user buffer could not all be sent on the
socket at once.  There are two kinds of situation to handle

1) User code handles it: The connection only has extensions active that do not rewrite the buffer.
In this case, the patch caused libwebsocket_write() to simply return the amount of user buffer that
was consumed (this is specifically the amount of user buffer used in sending what was accepted,
nothing else).  So user code can just advance its buffer that much and resume sending when the socket
is writable again.  This continues the frame rather than starting a new one or new fragment.

2) The connections has extensions active which actually send something quite different than what the
user buffer contains, for example a compression extension.  In this case, libwebsockets will dynamically
malloc a buffer to contain a copy of the remaining unsent data, request notifiction when writeable again,
and automatically spill and free this buffer with the highest priority before passing on the writable
notification to anything else.  For this situation, the call to write will return that it used the
whole user buffer, even though part is still rebuffered.

This patch should enable libwebsockets to detect the two cases and take the appropriate action.

There are also two choices for user code to deal with partial sends.

1) Leave the no_buffer_all_partial_tx member in the protocol struct at zero.  The library will dyamically
buffer anything you send that did not get completely written to the socket, and automatically spill it next
time the socket is writable.  You can use this method if your sent frames are relatvely small and unlikely to get
truncated anyway.

2) Set the no_buffer_all_partial_tx member in the protocol struct.  User code now needs to take care of the
return value from libwebsocket_write() and deal with resending the remainder if not all of the requested amount
got sent.  You should use this method if you are sending large messages and want to maximize throughput and efficiency.

Since the new member no_buffer_all_partial_tx will be zero by default, this patch will auto-rebuffer any
partial sends by default.  That's good for most cases but if you attempt to send large blocks, make sure you
follow option 2) above.

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 40 client connection properly nonblocking

As spotted by JM on Trac#40

http://libwebsockets.org/trac/libwebsockets/ticket/40

client connect didn't do anything about being truly nonblocking.  This patch
should hopefully solve that.

Signed-off-by: Andy Green <andy.green@linaro.org>

force failed header parse to act

''...Websocket servers return failure responses other than HTTP Status 101 in the case of
mismatches of WebSocket version or additional header data etc...
It seems that libwebsockets shows "WARN: problems parsing header" error in such cases without
calling any callbacks or returning error code. Is this right?
I modified lib/client.c to handle this.''

Signed-off-by: Fujii Bunichiroh <Bunichiroh.Fujii@jp.sony.com>

remove bae64 decode and selftest

Signed-off-by: Andy Green <andy.green@linaro.org>

trac #39: allow fd 0 for connection

Reported (with fix) by Martijn The http://libwebsockets.org/trac/ticket/39

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake fix for osx 64 bit

AG tested on 64-bit Fedora no problems

Signed-off-by: Erland Isaksson <erland@isaksson.info>

trac 36 make libwebsocket_set_timeout public

Signed-off-by: Andy Green <andy.green@linaro.org>

change LWS_CALLBACK_FILTER_PROTOCOL_CONNECTION user param usage

Also audit the bail_nuke_ah usage as Daniel Griscom suggested.

Signed-off-by: Andy Green <andy.green@linaro.org>

fix http incomplete send handling

Signed-off-by: David Gauchard <gauchard@laas.fr>

Request "send" not to trigger SIGPIPE on errors.

add CMake find function for libwebsockets

Signed-off-by: Jiri Hnidek <jiri.hnidek@tul.cz>

Added additional LWS_CALLBACK_CLOSED_HTTP callback to just_kill_connection branch of close_and_free_session

Support empty ping and pong packets.

Declare "sha1_init" statically to avoid naming conflicts if linked statically with other libraries providing a method with the same name (e.g. OpenSSL).

add LWS_CALLBACK_CLOSED_HTTP

Adds a callback to allow handling HTTP protocol connections
closing.

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 28 - differentiate between http serving and accepted http connection

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 28 brackets around POLLIN flag test

Signed-off-by: Simon Wulf <simonwulf.lund@bredband.net>

log err if unable to open http file

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 29 convert more snprintf stowaways

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 29 echo remoe mention of syslog constants for win32

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 29 avoid syslog in echo test if win32

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 29 eliminate snprintf again

Signed-off-by: Andy Green <andy.green@linaro.org>

trac 29 win32 EWOULDBLOCK

Signed-off-by: Andy Green <andy.green@linaro.org>

Trac 29: win32 fix LWS_VISIBLE issue

Reported-by: davetaflin
Signed-off-by: Andy Green <andy.green@linaro.org>

cygwin compatibility

Signed-off-by: Bob Roberts <bobroberts177@gmail.com>

trac 27: use fullpath in OSX dynamic libs

Signed-off-by: thinkski <chiszp@gmail.com>

pkg config avoid hardcoded prefix

Signed-off-by: thinkski <chiszp@gmail.com>

cmake use existing spec file for RPM generation

Signed-off-by: Tomas Cejka <cejkat@cesnet.cz>

cmake added RPM generation

Signed-off-by: Tomas Cejka <cejkat@cesnet.cz>

cmake improve cross documentation

Signed-off-by: Andy Green <andy.green@linaro.org>

Hide private library symbols

Signed-off-by: Peter Pentchev <roam@ringlet.net>

SOL_SNDBUF set to protocol rx size

This tells the OS to reserve a TX buffer at least the size of the biggest RX frame
expected, for both server and client connections.

In Linux, the OS reserves 2 x the requested amount.

This is aimed at reducing the partial large atomic frame send problem to the point
it's only coming at large atomic frames the OS balks at reserving the size for.

If you have a lot of data to send, it is better to split it into multiple writes,
and use the FIN / CONTINUATION websocket stuff to manage it.

See the "fraggle" test app for example code of how to do that.

Signed-off-by: Andy Green <andy.green@linaro.org>

test server improve return checking

Signed-off-by: Andy Green <andy.green@linaro.org>

cosmetic debug improvements

Signed-off-by: Andy Green <andy.green@linaro.org>

BUGFIX: Properly add libcyassl to the lib list.

This would cause failure to build the test apps with CyaSSL.

Check for GCC/G++ explicitly instead of simply Unix.

Since we're setting gcc specific build flags.

Allow setting the resource path via command line.

It is awfully limiting when you have to set the resource path at compile time.

Instead also allow setting it via the --resource_path command line option.

introduce api for unthrottle all connections of protocol

Signed-off-by: Andy Green <andy.green@linaro.org>

test client have u option also reject deflate stream

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake add gcc compile options

Signed-off-by: Andy Green <andy.green@linaro.org>

rxflow remove recursion and simplify

Signed-off-by: Andy Green <andy.green@linaro.org>

replaced last remaining close on wsi->sock with compatible_close

Signed-off-by: Tobias Maier <tobias.maier@netplace.com>

Fixed read of the jpg in Windows

Windows needs to open things with _O_BINARY

Signed-off-by: Michel Archambault <marchamb@matrox.com>

update service_fd comment and api docs

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake update specfile

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake add missing echo test app

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake packageconfig installs to lib dir

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake ignore non source things for make dist

Signed-off-by: Andy Green <andy.green@linaro.org>

android include gregorys example makefile

Signed-off-by: Gregory Junker <ggjunker@gmail.com>

update version 1.3 soname 4

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake some cosmetic improvementss

Signed-off-by: Edwin van den Oetelaar <oetelaar.automatisering@gmail.com>

autotools remove INSTALL m4 COPYING

Also get rid of m4/ and COPYING, it's a copy of LGPL2.1 but without the
exceptions we added in LICENSE.

Signed-off-by: Edwin van der Oetelaar <oetelaar.automatisering@gmail.com>
Signed-off-by: Andy Green <andy.green@linaro.org>

remove autotools build

Signed-off-by: Andy Green <andy.green@linaro.org>

document revent clearing scheme

Signed-off-by: Andy Green <andy.green@linaro.org>

clear down wsi_lookup on allocation and clear pollfd revents if serviced

This should get rid of a valgrind uninitialized report when using extpoll,
and gives a new way to share the poll loop with external sockets.

If a pollfd says it has something, you can just pass it to
libwebsocket_serice_fd() whether it is a socket handled by lws or not.
If it sees it is a lws socket, the traffic will be handled and
pollfd->revents will be zeroed now.

If the socket is foreign to lws, it leaves revents alone.  So you can see
if you should service by checking the pollfd revents after letting
lws try to service it.

Signed-off-by: Andy Green <andy.green@linaro.org>

reduce httpd close debug spew

Signed-off-by: Andy Green <andy.green@linaro.org>

cmake fix install path

Signed-off-by: Andy Green <andy.green@linaro.org>

correct example lws_write in header

Signed-off-by: Alex Rhatyshnyak <a.rhatushnyak@klervi.com>

dont leak when protocol agreement probs

Signed-off-by: Andy Green <andy.green@linaro.org>

Fix printf format error

Signed-off-by: Henrik Abelsson <henrik@abelsson.com>

cmake win32 change helper includes to HDR_PUBLIC so they install

Signed-off-by: Andy Green <andy.green@linaro.org>

dont sweat unkknown method for client

Signed-off-by: Andy Green <andy.green@linaro.org>

POLLOUT during wait for server response not an error

Signed-off-by: Andy Green <andy.green@linaro.org>

fix win32helpers gettimeofday epoch

Signed-off-by: Arokux B <arokux@gmail.com>

trace 22 fix zero length close from client

close() from javascript api in Chrome and Firefox doesn't do the
right thing.  It's because the payload is zero-length (with a frame
key...)  This fixes it.

Reported-by: 巫书轶
Signed-off-by: Andy Green <andy.green@linaro.org>

cmake mingw no need for websock w32

2.  Comment out this line

http://git.warmcat.com/cgi-bin/cgit/libwebsockets/tree/CMakeLists.txt#n279

the file websock-w32.c is already compiled indirectly from libwebsockets.c

Signed-off-by: Arokux B <arokux@gmail.com>

normalize formatting in gettimeofday

Signed-off-by: Andy Green <andy.green@linaro.org>

fix win32 gettimeofday for mingw

here are the problems that I found while compiling libwebsockets with
CMake using as a generator MinGW Makefiles.

1. In http://git.warmcat.com/cgi-bin/cgit/libwebsockets/plain/win32port/win32helpers/gettimeofday.c

Remove spaces around time.h, otherwise it looks for " time.h " and
fails with "No such file ..."

I also needed to comment out

struct timezone
{
  int  tz_minuteswest; /* minutes W of Greenwich */
  int  tz_dsttime;     /* type of dst correction */
};

It is already declared in mingw's time.h

Signed-off-by: Arokux B <arokux@gmail.com>

cmake lib lib64 problem

As discussed here, there's a semi-standardized way to inform
cmake that the install target wants its libs in .../lib64

https://code.google.com/p/bullet/issues/detail?id=174

this patch adds the necessary fiddling

Signed-off-by: Andy Green <andy.green@linaro.org>

fix mime type on leaf header

Signed-off-by: Luc <luc@honk-honk.com>

reflect send completeness in lws_write return

under load, writing packet sizes to the socket that are normally fine
can do partial writes, eg asking to write 4096 may only take 2800 of
it and return 2800 from the actual send.

Until now lws assumed that if it was safe to send, it could take any
size buffer, that's not the case under load.

This patch changes lws_write to return the amount actually taken...
that and the meaning of it becomes tricky when dealing with
compressed links, the amount taken and the amount sent differ.  Also
there is no way to recover at the moment from a protocol-encoded
frame only being partially accepted... however for http file send
content it can and does recover now.

Small frames don't have to take any care about it but large atomic
sends (> 2K) have been seen to fail under load.

Signed-off-by: Andy Green <andy.green@linaro.org>

move ssl cipher list from compiletime to context creation time option

More flexible this way... NULL for the new member means use
the ssl library default set of ciphers, so as long as your info
struct is zerod by bss or memset, you don't need to do anything
about this change unless you want to set the cipher list.

Signed-off-by: Andy Green <andy.green@linaro.org>

remove MAX_HEADER_NAME_LENGTH from header

Still lurking aound as a ghost after all real usess of it
were removed with the minilex changes.

Signed-off-by: Andy Green <andy.green@linaro.org>

Fix running test programs from within visual studio.

Copy test-server data to a path so that we can run the test-server from
within visual studio without having to copy stuff around.

Print SSL error codes as well as the string.

CYASSL doesn't compile with the error strings in some configurations, so
it is nice to have the error code on SSL errors as well.

Set the _DEBUG macro for CMake also.

Fixed ssl cert generation on Windows.

Added CPack support + some more.

- "make dist" will now produce a tar.gz file. This includes everything (not really what we want but a start).
- Got rid of a bunch of defines and variables that aren't used.
- Added the option LINK_TESTAPPS_DYNAMIC that enables linking to the shared library version of the lib for the test apps, instead of doing it statically like the default is.
- Fixed proper support for the --prefix stuff (-DCMAKE_INSTALL_PREFIX for cmake). (Don't specify /usr/local explicitly, that is the default anyway and will break other platforms).

- Note: I noticed a problem with the "INSTALL_DATADIR" define used by the test-apps. Since we hard code the path to the certs using this, doing "DESTDIR=/bla make install" will result in not being able to use the SSL mode for the test-apps since they won't find the certs. (This also applies to the autoconf project). Fixed this by setting "../share" as the default location instead of using the prefix.

BUGFIX: Fixed bug for installing test-apps for CMake.

Appending to the TEST_APP_LIST variable function did not work from within the create_test_app function (It would create a function scope variable with the same name and discard it). We instead need to use a macro for this.

The previous commit also removed the duplicate install for the test-apps (because we want all the install steps in one place in the file for clarity). That change made it obvious that the TEST_APP_LIST never gets populated.

Fixed CMake compile options.

Fixed so that the build options for the CMake project works:

- The test apps used the LWS_NO_EXTENSIONS define, so they needed lws_config.h included when building using CMake.
- Rename some options so that individual test apps can be turned off.
- Separate building the test-client/test-server and compiling the server/client parts into the lib.
- Don't include server or client specific sources into the build if they shouldn't be built.
- Added an error if both client and server parts are excluded at the same time (makes no sense).
- Removed duplicate install targets for the test apps.
- Commented out the WITH_LIBCRYPTO option since it isn't used at the moment.

getifaddrs missing more user friendly in CMake.

Don't require the user to enable using the built-in BSD getifaddrs implementation on systems such as uclibc that lacks it manually.

Instead if getifaddrs doesn't exist, use the BSD one automatically, except if the user explicitly tells the user not to do this using WITHOUT_BUILTIN_GETIFADDRS (which will result in a compilation error, but at least with a nice error message explaining why).

Fix LWS_NO_SERVER build.

"int n" was used also when server parts are not compiled, so don't put it inside the LWS_NO_SERVER ifdef.

Bugfix compiling for cross compiling.

! does not mean NOT, use NOT instead :)

keepalive swap interval and probes

The interval and probes members for setting keepalive options
per-connection on Linux were swapped.

Signed-off-by: Larry Hayes <lhayesg@gmail.com>

provide socketfd at in param for LWS_CALLBACK_FILTER_NETWORK_CONNECTION

Signed-off-by: Edwin van den Oetelaar <oetelaar.automatisering@gmail.com>
Signed-off-by: Andy Green <andy.green@linaro.org>

trac 18 deal with service connection timing out

While looking at http://libwebsockets.org/trac/ticket/18
noticed the flow for timeout in service_fd will do bad things
if the fd we came to service has timed out.  It gets freed and
then "serviced'.

Reported-by: Joakim Soderberg <joakim.soderberg@gmail.com>
Signed-off-by: Andy Green <andy.green@linaro.org>

cmake fix for older cmake

AG this wasn't needed on cmake 2.8.10.2 but did not make any
problem either

Signed-off-by: Edwin van den Oetelaar <oetelaar.automatisering@gmail.com>

update missed extpoll calls to use correct args for ssl

http://libwebsockets.org/trac/ticket/15#comment:16

Reported-by: Joakim Soderberg <joakim.soderberg@gmail.com>
Signed-off-by: Andy Green <andy.green@linaro.org>

cmake docs add note about prefix

Signed-off-by: Andy Green <andy.green@linaro.org>

Create the "doc" dir before generating docs.

The documentation wouldn't be generated if the path didn't exist.

Added "make install" support to the CMake project.

- Does everything as the autoconf script does (I think).
- Generates SSL certs for the test-server
- Installs all test apps.
- Defaults to /usr/local/
- "DESTDIR=/bla make install" works
- Append "libwebsockets" to start of the test-app names like the autoconf stuff does.
- Only tested on OSX so far...

trac 17 update set_log_level api notice

http://libwebsockets.org/trac/ticket/17

Reported-by: dbrnz
Signed-off-by: Andy Green <andy.green@linaro.org>

move cms cmake addition to cmake module paths earlier

Signed-off-by: Edwin van den Oetelaar <oetelaar.automatisering@gmail.com>

add FindGit.cmake

Signed-off-by: Andy Green <andy.green@linaro.org>

document ensure_user_space going private

Signed-off-by: Andy Green <andy.green@linaro.org>

remove lws_ensure_user_space from public api change return

The function has a logical problem when the size of the requested
allocation is 0, it will return NULL which is overloaded as
failure.

Actually the whole function is evil as an api, this patch moves
it out of the public API space and fixes it to return 0 for
success or 1 for fail.  Private code does not need to to return
wsi->user_space and public code should only get that from the
callback as discussed on trac recently.

Thanks to Edwin for debugging the problem.

Reported-by: Edwin van den Oetelaar <oetelaar.automatisering@gmail.com>
Signed-off-by: Andy Green <andy.green@linaro.org>

update arm build stats

Clean out the historical numbers, update info

Signed-off-by: Andy Green <andy.green@linaro.org>

trim proxy ads plus hostname stg

128 should be fine for hostname and proxy ads

Signed-off-by: Andy Green <andy.green@linaro.org>

restrict http send buffer to 4096

send() is able to complete but not send everything asked...
we don't deal with that yet.  On an ARM9 board, leaf.jpg
failed because the send() only accepted 7240 of 8192... not
a failure but it implies reporting partial send to the
caller.  Work around by keeping the send buffer at 4096.

Signed-off-by: Andy Green <andy.green@linaro.org>