/*
 * Copyright 2011 The Netty Project
 *
 * The Netty Project licenses this file to you under the Apache License,
 * version 2.0 (the "License"); you may not use this file except in compliance
 * with the License. You may obtain a copy of the License at:
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
 * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
 * License for the specific language governing permissions and limitations
 * under the License.
 */
package io.netty.handler.codec.http.websocketx;

import io.netty.channel.Channel;
import io.netty.handler.codec.http.DefaultHttpResponse;
import io.netty.handler.codec.http.HttpRequest;
import io.netty.handler.codec.http.HttpResponse;
import io.netty.handler.codec.http.HttpResponseStatus;
import io.netty.handler.codec.http.HttpVersion;
import io.netty.handler.codec.http.HttpHeaders.Names;

/**
 * Instances the appropriate handshake class to use for clients
 */
public class WebSocketServerHandshakerFactory {

	private final String webSocketURL;

	private final String subProtocols;

	private boolean allowExtensions = false;

	/**
	 * Constructor specifying the destination web socket location
	 * 
	 * @param webSocketURL
	 *            URL for web socket communications. e.g "ws://myhost.com/mypath". Subsequent web socket frames will be
	 *            sent to this URL.
	 * @param subProtocols
	 *            CSV of supported protocols. Null if sub protocols not supported.
	 * @param allowExtensions
	 *            Allow extensions to be used in the reserved bits of the web socket frame
	 */
	public WebSocketServerHandshakerFactory(String webSocketURL, String subProtocols, boolean allowExtensions) {
		this.webSocketURL = webSocketURL;
		this.subProtocols = subProtocols;
		this.allowExtensions = allowExtensions;
	}

	/**
	 * Instances a new handshaker
	 * 
	 * @return A new WebSocketServerHandshaker for the requested web socket version. Null if web socket version is not
	 *         supported.
	 */
	public WebSocketServerHandshaker newHandshaker(HttpRequest req) {

		String version = req.getHeader(Names.SEC_WEBSOCKET_VERSION);
		if (version != null) {
			if (version.equals(WebSocketVersion.V13.toHttpHeaderValue())) {
				// Version 13 of the wire protocol - RFC 6455 (version 17 of the draft hybi specification).
				return new WebSocketServerHandshaker13(webSocketURL, subProtocols, this.allowExtensions);
			} else if (version.equals(WebSocketVersion.V08.toHttpHeaderValue())) {
				// Version 8 of the wire protocol - version 10 of the draft hybi specification.
				return new WebSocketServerHandshaker08(webSocketURL, subProtocols, this.allowExtensions);
			} else {
				return null;
			}
		} else {
			// Assume version 00 where version header was not specified
			return new WebSocketServerHandshaker00(webSocketURL, subProtocols);
		}
	}

	/**
	 * Return that we need cannot not support the web socket version
	 * 
	 * @param channel
	 *            Channel
	 */
	public void sendUnsupportedWebSocketVersionResponse(Channel channel) {
		HttpResponse res = new DefaultHttpResponse(HttpVersion.HTTP_1_1, new HttpResponseStatus(101,
				"Switching Protocols"));
		res.setStatus(HttpResponseStatus.UPGRADE_REQUIRED);
		res.setHeader(Names.SEC_WEBSOCKET_VERSION, "13");
		channel.write(res);
	}

}

@@ -3415,12 +3415,48 @@
 ON, 
-%2213%22
+WebSocketVersion.V13.toHttpHeaderValue()
 );%0A%09

Use WebSocketVersion enum rather than string literal