Skip to content

Fixed a test#15

Open
bogdanmaxim wants to merge 14 commits intonmosafi:masterfrom
bogdanmaxim:master
Open

Fixed a test#15
bogdanmaxim wants to merge 14 commits intonmosafi:masterfrom
bogdanmaxim:master

Conversation

@bogdanmaxim
Copy link
Copy Markdown

No description provided.

Wintermoose and others added 14 commits October 4, 2010 11:50
Enable cross-site requests
a) support OPTIONS request with Access-Control-* headers, which works with newever browsers
b) support callback polling transport (GET+jsonp), which will be used by older browsers

The access is controlled by CometHttpHandler.AllowOrigin = "*";, where the value can be null, asterix, or comma-separated list of domains. The chat has been modified to demonstrate this (see the if block after configure section in chat.js)

In addition, the polling duration and client tiemouts have been made configurable.

Note: the long-polling vs callback detection is not 100% proper, as we look only at the method used. This will however work for clients following the "SHOULD" recommendations in Bayeux specifications.
Conflicts:
	src/Samples/Chat/Global.asax.cs
…nnects It should_not_publish_the_disconnected_event_again test
Moved to VS 2010 solutions and cleaned up old version and added a copy of The Bayeux Protocol for reference.  Also, fixed a couple of issues to do with specifications.  Some specifications are still failing, particularly in relation to callback messaging.
Conflicts:
	src/AspComet/MessageConverter.cs
The regular expression that removed nulls from JSON messages would leave behind a comma at the beginning of the top level of the message if the first property was null but there were subsequent properties.
Changed remaining project names to remove vs2010 label.  Also implemented checks for subscribe and unsubscribe message handlers so that they respond with a 402 error indicating that the client is unknown.
@nmosafi nmosafi added the ready label Oct 20, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants