update structure according to proper go project

This commit is contained in:
Adrien Marquès 2018-09-29 14:36:47 +02:00
parent 53de261a43
commit a9986c3123
10 changed files with 356 additions and 399 deletions

View File

@ -1,13 +1,13 @@
package ws
package websocket
import (
"time"
"sync"
"bufio"
"encoding/binary"
"git.xdrm.io/gws/internal/http/upgrade/request"
"net"
"fmt"
"git.xdrm.io/go/websocket/internal/http/upgrade/request"
"net"
"sync"
"time"
)
// Represents a client socket utility (reader, writer, ..)
@ -22,7 +22,7 @@ type clientIO struct {
}
// Represents all channels that need a client
type clientChannelSet struct{
type clientChannelSet struct {
receive chan Message
send chan Message
}
@ -35,11 +35,8 @@ type client struct {
status MessageError // close status ; 0 = nothing ; else -> must close
}
// Create creates a new client
func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*client, error){
func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*client, error) {
/* (1) Manage UPGRADE request
---------------------------------------------------------*/
@ -60,7 +57,6 @@ func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*cli
return nil, fmt.Errorf("Upgrade error (HTTP %d)\n", res.GetStatusCode())
}
/* (2) Initialise client
---------------------------------------------------------*/
/* (1) Get upgrade data */
@ -77,7 +73,7 @@ func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*cli
iface: &Client{
Protocol: string(clientProtocol),
Arguments: [][]string{ []string{ clientURI } },
Arguments: [][]string{[]string{clientURI}},
},
ch: clientChannelSet{
@ -86,12 +82,10 @@ func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*cli
},
}
/* (3) Find controller by URI
---------------------------------------------------------*/
/* (1) Try to find one */
controller, arguments := ctl.Match(clientURI);
controller, arguments := ctl.Match(clientURI)
/* (2) If nothing found -> error */
if controller == nil {
@ -101,8 +95,6 @@ func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*cli
/* (3) Copy arguments */
cli.iface.Arguments = arguments
/* (4) Launch client routines
---------------------------------------------------------*/
/* (1) Launch client controller */
@ -123,11 +115,8 @@ func buildClient(s net.Conn, ctl ControllerSet, serverCh serverChannelSet) (*cli
}
// reader reads and parses messages from the buffer
func clientReader(c *client){
func clientReader(c *client) {
var frag *Message
closeStatus := NORMAL
@ -149,7 +138,9 @@ func clientReader(c *client){
if err == ErrUnmaskedFrame || err == ErrReservedBits {
closeStatus = PROTOCOL_ERR
}
if err != nil { break }
if err != nil {
break
}
/* (3) Fail on invalid message */
msgErr := msg.check(frag != nil)
@ -184,7 +175,9 @@ func clientReader(c *client){
mustClose = true
}
if mustClose { break }
if mustClose {
break
}
}
@ -248,11 +241,9 @@ func clientReader(c *client){
}
// writer writes into websocket
// and is triggered by client.ch.send channel
func clientWriter(c *client){
func clientWriter(c *client) {
c.io.writing = true // if channel still exists
@ -278,14 +269,11 @@ func clientWriter(c *client){
}
// closes the connection
// send CLOSE frame is 'status' is not NONE
// wait for the next message (CLOSE acknowledge) if 'clientACK'
// then delete client
func (c *client) close(status MessageError, clientACK bool){
func (c *client) close(status MessageError, clientACK bool) {
/* (1) Fail if already closing */
alreadyClosing := false
@ -304,11 +292,9 @@ func (c *client) close(status MessageError, clientACK bool){
}
/* (3) kill reader if still running */
c.io.sock.SetReadDeadline(time.Now().Add(time.Second*-1))
c.io.sock.SetReadDeadline(time.Now().Add(time.Second * -1))
c.io.reading.Wait()
if status != NONE {
/* (3) Build message */
@ -328,7 +314,6 @@ func (c *client) close(status MessageError, clientACK bool){
}
/* (2) Wait for client CLOSE if needed */
if clientACK {
@ -358,4 +343,3 @@ func (c *client) close(status MessageError, clientACK bool){
return
}

View File

@ -1,13 +1,12 @@
package main
package iface
import (
"git.xdrm.io/gws/ws"
"time"
"fmt"
ws "git.xdrm.io/go/websocket"
"time"
)
func main(){
func main() {
startTime := time.Now().UnixNano()
@ -15,10 +14,10 @@ func main(){
serv := ws.CreateServer("0.0.0.0", 4444)
/* (2) Bind default controller */
serv.BindDefault(func(cli *ws.Client, receiver <-chan ws.Message, sender chan<- ws.Message, bc chan<- ws.Message){
serv.BindDefault(func(cli *ws.Client, receiver <-chan ws.Message, sender chan<- ws.Message, bc chan<- ws.Message) {
defer func(){
if (recover() != nil) {
defer func() {
if recover() != nil {
fmt.Printf("*** PANIC\n")
}
}()
@ -34,11 +33,11 @@ func main(){
})
/* (3) Bind to URI */
err := serv.Bind("/channel/./room/./", func(cli *ws.Client, receiver <-chan ws.Message, sender chan<- ws.Message, bc chan<- ws.Message){
err := serv.Bind("/channel/./room/./", func(cli *ws.Client, receiver <-chan ws.Message, sender chan<- ws.Message, bc chan<- ws.Message) {
fmt.Printf("[uri] connected\n")
for msg := range receiver{
for msg := range receiver {
fmt.Printf("[uri] received '%s'\n", msg.Data)
sender <- msg
@ -48,7 +47,9 @@ func main(){
fmt.Printf("[uri] unexpectedly closed\n")
})
if err != nil { panic(err) }
if err != nil {
panic(err)
}
/* (4) Launch the server */
err = serv.Launch()
@ -57,7 +58,6 @@ func main(){
return
}
fmt.Printf("+ elapsed: %1.1f us\n", float32(time.Now().UnixNano()-startTime)/1e3)
}

View File

@ -1,7 +1,7 @@
package ws
package websocket
import (
"git.xdrm.io/gws/internal/uri/parser"
"git.xdrm.io/go/websocket/internal/uri/parser"
)
// Represents available information about a client
@ -26,14 +26,12 @@ type ControllerSet struct {
Uri []*Controller // uri controllers
}
// Match finds a controller for a given URI
// also it returns the matching string patterns
func (s *ControllerSet) Match(uri string) (*Controller, [][]string){
func (s *ControllerSet) Match(uri string) (*Controller, [][]string) {
/* (1) Initialise argument list */
arguments := [][]string{ []string{ uri } }
arguments := [][]string{[]string{uri}}
/* (2) Try each controller */
for _, c := range s.Uri {

View File

@ -1,15 +1,13 @@
package request
import (
"git.xdrm.io/gws/internal/http/upgrade/response"
"git.xdrm.io/gws/internal/http/upgrade/request/parser/header"
"fmt"
"git.xdrm.io/go/websocket/internal/http/upgrade/request/parser/header"
"git.xdrm.io/go/websocket/internal/http/upgrade/response"
"strconv"
"strings"
)
// checkHost checks and extracts the Host header
func (r *T) extractHostPort(bb header.HeaderValue) error {
@ -52,12 +50,13 @@ func (r *T) extractHostPort(bb header.HeaderValue) error {
}
// checkOrigin checks the Origin Header
func (r *T) extractOrigin(bb header.HeaderValue) error {
// bypass
if bypassOriginPolicy { return nil }
if bypassOriginPolicy {
return nil
}
if len(bb) != 1 {
r.code = response.FORBIDDEN
@ -92,7 +91,7 @@ func (r *T) checkConnection(bb header.HeaderValue) error {
for _, b := range bb {
if strings.ToLower( string(b) ) == "upgrade" {
if strings.ToLower(string(b)) == "upgrade" {
r.hasConnection = true
return nil
}
@ -113,7 +112,7 @@ func (r *T) checkUpgrade(bb header.HeaderValue) error {
return &InvalidRequest{"Upgrade", fmt.Sprintf("expected single value, got %d", len(bb))}
}
if strings.ToLower( string(bb[0]) ) == "websocket" {
if strings.ToLower(string(bb[0])) == "websocket" {
r.hasUpgrade = true
return nil
}
@ -152,7 +151,6 @@ func (r *T) extractKey(bb header.HeaderValue) error {
}
// extractProtocols extracts the 'Sec-WebSocket-Protocol' header
// it can contain multiple values
func (r *T) extractProtocols(bb header.HeaderValue) error {

View File

@ -1,12 +1,11 @@
package request
import (
"git.xdrm.io/gws/internal/http/upgrade/response"
"fmt"
"git.xdrm.io/gws/internal/http/upgrade/request/parser/header"
"git.xdrm.io/go/websocket/internal/http/upgrade/request/parser/header"
"git.xdrm.io/go/websocket/internal/http/upgrade/response"
)
// parseHeader parses any http request line
// (header and request-line)
func (r *T) parseHeader(b []byte) error {
@ -27,8 +26,6 @@ func (r *T) parseHeader(b []byte) error {
}
/* (2) Other lines -> Header-Name: Header-Value
---------------------------------------------------------*/
/* (1) Try to parse header */
@ -40,19 +37,25 @@ func (r *T) parseHeader(b []byte) error {
/* (2) Manage header */
switch head.Name {
case header.HOST: err = r.extractHostPort(head.Values)
case header.ORIGIN: err = r.extractOrigin(head.Values)
case header.UPGRADE: err = r.checkUpgrade(head.Values)
case header.CONNECTION: err = r.checkConnection(head.Values)
case header.WSVERSION: err = r.checkVersion(head.Values)
case header.WSKEY: err = r.extractKey(head.Values)
case header.WSPROTOCOL: err = r.extractProtocols(head.Values)
case header.HOST:
err = r.extractHostPort(head.Values)
case header.ORIGIN:
err = r.extractOrigin(head.Values)
case header.UPGRADE:
err = r.checkUpgrade(head.Values)
case header.CONNECTION:
err = r.checkConnection(head.Values)
case header.WSVERSION:
err = r.checkVersion(head.Values)
case header.WSKEY:
err = r.extractKey(head.Values)
case header.WSPROTOCOL:
err = r.extractProtocols(head.Values)
default:
return nil
}
// dispatch error
if err != nil {
return err
@ -62,8 +65,6 @@ func (r *T) parseHeader(b []byte) error {
}
// isComplete returns whether the Upgrade Request
// is complete (no missing required item)
func (r T) isComplete() error {

View File

@ -1,9 +1,9 @@
package request
import (
"git.xdrm.io/gws/internal/http/upgrade/response"
"git.xdrm.io/gws/internal/http/reader"
"fmt"
"git.xdrm.io/go/websocket/internal/http/reader"
"git.xdrm.io/go/websocket/internal/http/upgrade/response"
"io"
)
@ -14,7 +14,6 @@ func Parse(r io.Reader) (request *T, err error) {
req := new(T)
req.code = 500
/* (1) Parse request
---------------------------------------------------------*/
/* (1) Get chunk reader */
@ -50,23 +49,18 @@ func Parse(r io.Reader) (request *T, err error) {
return req, err
}
req.code = response.SWITCHING_PROTOCOLS
return req, nil
}
// StatusCode returns the status current
func (r T) StatusCode() response.StatusCode {
return r.code
}
// BuildResponse builds a response.T from the request
func (r *T) BuildResponse() *response.T{
func (r *T) BuildResponse() *response.T {
inst := new(response.T)
@ -84,9 +78,7 @@ func (r *T) BuildResponse() *response.T{
return inst
}
// GetURI returns the actual URI
func (r T) GetURI() string{
func (r T) GetURI() string {
return r.request.GetURI()
}

View File

@ -1,6 +1,6 @@
package request
import "git.xdrm.io/gws/internal/http/upgrade/response"
import "git.xdrm.io/go/websocket/internal/http/upgrade/response"
// If origin is required
const bypassOriginPolicy = true

View File

@ -1,10 +1,10 @@
package ws
package websocket
import (
"unicode/utf8"
"encoding/binary"
"fmt"
"io"
"encoding/binary"
"unicode/utf8"
)
var ErrUnmaskedFrame = fmt.Errorf("Received unmasked frame")
@ -18,7 +18,6 @@ var ErrInvalidOpCode = fmt.Errorf("Received invalid OpCode")
var ErrReservedBits = fmt.Errorf("Received reserved bits")
var CloseFrame = fmt.Errorf("Received close Frame")
// Maximum Header Size = Final/OpCode + isMask/Length + Length + Mask
const maximumHeaderSize = 1 + 1 + 8 + 4
const maxWriteChunk = 0x7fff
@ -46,8 +45,7 @@ const (
CLOSE MessageType = 0x08
PING MessageType = 0x09
PONG MessageType = 0x0a
);
)
// Represents a websocket message
type Message struct {
@ -57,9 +55,8 @@ type Message struct {
Data []byte
}
// receive reads a message form reader
func readMessage(reader io.Reader) (*Message, error){
func readMessage(reader io.Reader) (*Message, error) {
var err error
var tmpBuf []byte
@ -71,46 +68,53 @@ func readMessage(reader io.Reader) (*Message, error){
/* (2) Byte 1: FIN and OpCode */
tmpBuf = make([]byte, 1)
err = readBytes(reader, tmpBuf)
if err != nil { return m, err }
if err != nil {
return m, err
}
// check reserved bits
if tmpBuf[0] & 0x70 != 0 {
if tmpBuf[0]&0x70 != 0 {
return m, ErrReservedBits
}
m.Final = bool( tmpBuf[0] & 0x80 == 0x80 )
m.Type = MessageType( tmpBuf[0] & 0x0f )
m.Final = bool(tmpBuf[0]&0x80 == 0x80)
m.Type = MessageType(tmpBuf[0] & 0x0f)
/* (3) Byte 2: Mask and Length[0] */
tmpBuf = make([]byte, 1)
err = readBytes(reader, tmpBuf)
if err != nil { return m, err }
if err != nil {
return m, err
}
// if mask, byte array not nil
if tmpBuf[0] & 0x80 == 0x80 {
if tmpBuf[0]&0x80 == 0x80 {
mask = make([]byte, 0)
}
// payload length
m.Size = uint( tmpBuf[0] & 0x7f )
m.Size = uint(tmpBuf[0] & 0x7f)
/* (4) Extended payload */
if m.Size == 127 {
tmpBuf = make([]byte, 8)
err := readBytes(reader, tmpBuf)
if err != nil { return m, err }
if err != nil {
return m, err
}
m.Size = uint( binary.BigEndian.Uint64(tmpBuf) )
m.Size = uint(binary.BigEndian.Uint64(tmpBuf))
} else if m.Size == 126 {
tmpBuf = make([]byte, 2)
err := readBytes(reader, tmpBuf)
if err != nil { return m, err }
if err != nil {
return m, err
}
m.Size = uint( binary.BigEndian.Uint16(tmpBuf) )
m.Size = uint(binary.BigEndian.Uint16(tmpBuf))
}
@ -119,7 +123,9 @@ func readMessage(reader io.Reader) (*Message, error){
tmpBuf = make([]byte, 4)
err := readBytes(reader, tmpBuf)
if err != nil { return m, err }
if err != nil {
return m, err
}
mask = make([]byte, 4)
copy(mask, tmpBuf)
@ -142,7 +148,7 @@ func readMessage(reader io.Reader) (*Message, error){
// {3} Unmask data //
if mask != nil {
for i, l := cursor, cursor+nbread ; i < l ; i++ {
for i, l := cursor, cursor+nbread; i < l; i++ {
mi := i % 4 // mask index
m.Data[i] = m.Data[i] ^ mask[mi]
@ -166,9 +172,6 @@ func readMessage(reader io.Reader) (*Message, error){
}
// Send sends a frame over a socket
func (m Message) Send(writer io.Writer) error {
@ -176,7 +179,7 @@ func (m Message) Send(writer io.Writer) error {
// fix size
if uint(len(m.Data)) <= m.Size {
m.Size = uint( len(m.Data) )
m.Size = uint(len(m.Data))
}
/* (1) Byte 0 : FIN + opcode */
@ -184,12 +187,12 @@ func (m Message) Send(writer io.Writer) error {
if !m.Final {
final = 0
}
header = append(header, final | byte(m.Type) )
header = append(header, final|byte(m.Type))
/* (2) Get payload length */
if m.Size < 126 { // simple
header = append(header, byte(m.Size) )
header = append(header, byte(m.Size))
} else if m.Size <= 0xffff { // extended: 16 bits
@ -210,7 +213,7 @@ func (m Message) Send(writer io.Writer) error {
}
/* (3) Build write buffer */
writeBuf := make([]byte, 0, len(header) + int(m.Size))
writeBuf := make([]byte, 0, len(header)+int(m.Size))
writeBuf = append(writeBuf, header...)
writeBuf = append(writeBuf, m.Data[0:m.Size]...)
@ -219,14 +222,16 @@ func (m Message) Send(writer io.Writer) error {
cursor := 0
for cursor < toWrite {
maxBoundary := cursor+maxWriteChunk
maxBoundary := cursor + maxWriteChunk
if maxBoundary > toWrite {
maxBoundary = toWrite
}
// Try to wrote (at max 1024 bytes) //
nbwritten, err := writer.Write(writeBuf[cursor:maxBoundary])
if err != nil { return err }
if err != nil {
return err
}
// Update cursor //
cursor += nbwritten
@ -236,14 +241,11 @@ func (m Message) Send(writer io.Writer) error {
return nil
}
// Check for message errors with:
// (m) the current message
// (fragment) whether there is a fragment in construction
// returns the message error
func (m *Message) check(fragment bool) error{
func (m *Message) check(fragment bool) error {
/* (1) Invalid first fragment (not TEXT nor BINARY) */
if !m.Final && !fragment && m.Type != TEXT && m.Type != BINARY {
@ -312,8 +314,6 @@ func (m *Message) check(fragment bool) error{
return nil
}
// readBytes reads from a reader into a byte array
// until the byte length is fully filled with data
// loops while there is no error

View File

@ -1,15 +1,14 @@
package ws
package websocket
import (
"io"
"bytes"
"io"
"testing"
)
func TestSimpleMessageReading(t *testing.T) {
cases := []struct{
cases := []struct {
Name string
ReadBuffer []byte
Expected Message
@ -17,57 +16,57 @@ func TestSimpleMessageReading(t *testing.T) {
}{
{ // FIN ; TEXT ; Unmasked -> error
"must fail on unmasked frame",
[]byte{0x81,0x05,0x68,0x65,0x6c,0x6c,0x6f},
[]byte{0x81, 0x05, 0x68, 0x65, 0x6c, 0x6c, 0x6f},
Message{},
ErrUnmaskedFrame,
},
{ // FIN ; TEXT ; Unmasked -> error
"must fail because of RSV bit 1 set",
[]byte{0x81 | 0x40,0x10,0x00,0x00,0x00,0x00},
[]byte{0x81 | 0x40, 0x10, 0x00, 0x00, 0x00, 0x00},
Message{},
ErrReservedBits,
},
{ // FIN ; TEXT ; Unmasked -> error
"must fail because of RSV bit 2 set",
[]byte{0x81 | 0x20,0x10,0x00,0x00,0x00,0x00},
[]byte{0x81 | 0x20, 0x10, 0x00, 0x00, 0x00, 0x00},
Message{},
ErrReservedBits,
},
{ // FIN ; TEXT ; Unmasked -> error
"must fail because of RSV bit 3 set",
[]byte{0x81 | 0x10,0x10,0x00,0x00,0x00,0x00},
[]byte{0x81 | 0x10, 0x10, 0x00, 0x00, 0x00, 0x00},
Message{},
ErrReservedBits,
},
{ // FIN ; TEXT ; hello
"simple hello text message",
[]byte{0x81,0x85,0x00,0x00,0x00,0x00,0x68,0x65,0x6c,0x6c,0x6f},
Message{ true, TEXT, 5, []byte("hello") },
[]byte{0x81, 0x85, 0x00, 0x00, 0x00, 0x00, 0x68, 0x65, 0x6c, 0x6c, 0x6f},
Message{true, TEXT, 5, []byte("hello")},
nil,
},
{ // FIN ; BINARY ; hello
"simple hello binary message",
[]byte{0x82,0x85,0x00,0x00,0x00,0x00,0x68,0x65,0x6c,0x6c,0x6f},
Message{ true, BINARY, 5, []byte("hello") },
[]byte{0x82, 0x85, 0x00, 0x00, 0x00, 0x00, 0x68, 0x65, 0x6c, 0x6c, 0x6f},
Message{true, BINARY, 5, []byte("hello")},
nil,
},
{ // FIN ; BINARY ; test unmasking
"unmasking test",
[]byte{0x82,0x88,0x01,0x02,0x03,0x04,0x10,0x20,0x30,0x40,0x50,0x60,0x70,0x80},
Message{ true, BINARY, 8, []byte{0x11,0x22,0x33,0x44,0x51,0x62,0x73,0x84} },
[]byte{0x82, 0x88, 0x01, 0x02, 0x03, 0x04, 0x10, 0x20, 0x30, 0x40, 0x50, 0x60, 0x70, 0x80},
Message{true, BINARY, 8, []byte{0x11, 0x22, 0x33, 0x44, 0x51, 0x62, 0x73, 0x84}},
nil,
},
{ // FIN=0 ; TEXT ;
"non final frame",
[]byte{0x01,0x82,0x00,0x00,0x00,0x00,0x01,0x02},
Message{ false, TEXT, 2, []byte{0x01,0x02} },
[]byte{0x01, 0x82, 0x00, 0x00, 0x00, 0x00, 0x01, 0x02},
Message{false, TEXT, 2, []byte{0x01, 0x02}},
nil,
},
}
for _, tc := range cases{
for _, tc := range cases {
t.Run(tc.Name, func(t *testing.T){
t.Run(tc.Name, func(t *testing.T) {
reader := bytes.NewBuffer(tc.ReadBuffer)
@ -108,10 +107,9 @@ func TestSimpleMessageReading(t *testing.T) {
}
func TestReadEOF(t *testing.T) {
cases := []struct{
cases := []struct {
Name string
ReadBuffer []byte
eof bool
@ -127,58 +125,58 @@ func TestReadEOF(t *testing.T) {
true, false,
}, {
"only opcode and 0 length",
[]byte{0x82,0x00},
[]byte{0x82, 0x00},
false, true,
}, {
"missing extended 16 bits length",
[]byte{0x82,126},
[]byte{0x82, 126},
true, false,
}, {
"incomplete extended 16 bits length",
[]byte{0x82,126, 0x00},
[]byte{0x82, 126, 0x00},
true, false,
}, {
"complete extended 16 bits length",
[]byte{0x82,126, 0x00, 0x00},
[]byte{0x82, 126, 0x00, 0x00},
false, true,
}, {
"missing extended 64 bits length",
[]byte{0x82,127},
[]byte{0x82, 127},
true, false,
}, {
"incomplete extended 64 bits length",
[]byte{0x82,127, 0x00},
[]byte{0x82, 127, 0x00},
true, false,
}, {
"incomplete extended 64 bits length",
[]byte{0x82,127, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00},
[]byte{0x82, 127, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00},
true, false,
}, {
"complete extended 64 bits length",
[]byte{0x82,127, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00},
[]byte{0x82, 127, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00},
false, true,
}, {
"missing mask",
[]byte{0x82,0x80},
[]byte{0x82, 0x80},
true, false,
}, {
"incomplete mask 1",
[]byte{0x82,0x80, 0x00},
[]byte{0x82, 0x80, 0x00},
true, false,
}, {
"incomplete mask 2",
[]byte{0x82,0x80, 0x00, 0x00, 0x00},
[]byte{0x82, 0x80, 0x00, 0x00, 0x00},
true, false,
},{
}, {
"complete mask",
[]byte{0x82,0x80, 0x00, 0x00, 0x00, 0x00},
[]byte{0x82, 0x80, 0x00, 0x00, 0x00, 0x00},
false, false,
},
}
for _, tc := range cases{
for _, tc := range cases {
t.Run(tc.Name, func(t *testing.T){
t.Run(tc.Name, func(t *testing.T) {
reader := bytes.NewBuffer(tc.ReadBuffer)
@ -186,7 +184,7 @@ func TestReadEOF(t *testing.T) {
if tc.eof {
if err != io.EOF{
if err != io.EOF {
t.Errorf("Expected EOF, got %v", err)
}
@ -207,69 +205,67 @@ func TestReadEOF(t *testing.T) {
}
func TestSimpleMessageSending(t *testing.T) {
m4b1 := make([]byte, 0x7e - 1)
m4b1 := make([]byte, 0x7e-1)
m4b2 := make([]byte, 0x7e)
m4b3 := make([]byte, 0x7e + 1)
m4b3 := make([]byte, 0x7e+1)
m16b1 := make([]byte, 0xffff - 1)
m16b1 := make([]byte, 0xffff-1)
m16b2 := make([]byte, 0xffff)
m16b3 := make([]byte, 0xffff + 1)
m16b3 := make([]byte, 0xffff+1)
cases := []struct{
cases := []struct {
Name string
Base Message
Expected []byte
}{
{
"simple hello text message",
Message{ true, TEXT, 5, []byte("hello") },
[]byte{0x81,0x05,0x68,0x65,0x6c,0x6c,0x6f},
Message{true, TEXT, 5, []byte("hello")},
[]byte{0x81, 0x05, 0x68, 0x65, 0x6c, 0x6c, 0x6f},
}, {
"simple hello binary message",
Message{ true, BINARY, 5, []byte("hello") },
[]byte{0x82,0x05,0x68,0x65,0x6c,0x6c,0x6f},
Message{true, BINARY, 5, []byte("hello")},
[]byte{0x82, 0x05, 0x68, 0x65, 0x6c, 0x6c, 0x6f},
}, {
"other simple binary message",
Message{ true, BINARY, 8, []byte{0x10,0x20,0x30,0x40,0x50,0x60,0x70,0x80} },
[]byte{0x82,0x08,0x10,0x20,0x30,0x40,0x50,0x60,0x70,0x80},
Message{true, BINARY, 8, []byte{0x10, 0x20, 0x30, 0x40, 0x50, 0x60, 0x70, 0x80}},
[]byte{0x82, 0x08, 0x10, 0x20, 0x30, 0x40, 0x50, 0x60, 0x70, 0x80},
}, {
"non final frame",
Message{ false, TEXT, 2, []byte{0x01,0x02} },
[]byte{0x01,0x02,0x01,0x02},
Message{false, TEXT, 2, []byte{0x01, 0x02}},
[]byte{0x01, 0x02, 0x01, 0x02},
}, {
"125 > normal length",
Message{ true, TEXT, uint(len(m4b1)), m4b1 },
append([]byte{0x81,0x7e-1}, m4b1...),
Message{true, TEXT, uint(len(m4b1)), m4b1},
append([]byte{0x81, 0x7e - 1}, m4b1...),
}, {
"126 > extended 16 bits length",
Message{ true, TEXT, uint(len(m4b2)), m4b2 },
append([]byte{0x81,126,0x00,0x7e}, m4b2...),
Message{true, TEXT, uint(len(m4b2)), m4b2},
append([]byte{0x81, 126, 0x00, 0x7e}, m4b2...),
}, {
"127 > extended 16 bits length",
Message{ true, TEXT, uint(len(m4b3)), m4b3 },
append([]byte{0x81,126,0x00,0x7e+1}, m4b3...),
Message{true, TEXT, uint(len(m4b3)), m4b3},
append([]byte{0x81, 126, 0x00, 0x7e + 1}, m4b3...),
}, {
"fffe > extended 16 bits length",
Message{ true, TEXT, uint(len(m16b1)), m16b1 },
append([]byte{0x81,126, 0xff, 0xfe}, m16b1...),
Message{true, TEXT, uint(len(m16b1)), m16b1},
append([]byte{0x81, 126, 0xff, 0xfe}, m16b1...),
}, {
"ffff > extended 16 bits length",
Message{ true, TEXT, uint(len(m16b2)), m16b2 },
append([]byte{0x81,126,0xff,0xff}, m16b2...),
Message{true, TEXT, uint(len(m16b2)), m16b2},
append([]byte{0x81, 126, 0xff, 0xff}, m16b2...),
}, {
"10000 > extended 64 bits length",
Message{ true, TEXT, uint(len(m16b3)), m16b3 },
append([]byte{0x81,127, 0x00,0x00,0x00,0x00,0x00,0x01,0x00,0x00,}, m16b3...),
Message{true, TEXT, uint(len(m16b3)), m16b3},
append([]byte{0x81, 127, 0x00, 0x00, 0x00, 0x00, 0x00, 0x01, 0x00, 0x00}, m16b3...),
},
}
for _, tc := range cases{
for _, tc := range cases {
t.Run(tc.Name, func(t *testing.T){
t.Run(tc.Name, func(t *testing.T) {
writer := new(bytes.Buffer)
@ -291,9 +287,6 @@ func TestSimpleMessageSending(t *testing.T) {
}
func TestMessageCheck(t *testing.T) {
type Case struct {
@ -303,7 +296,7 @@ func TestMessageCheck(t *testing.T) {
Expected error
}
cases := []struct{
cases := []struct {
Name string
Cases []Case
}{
@ -358,7 +351,7 @@ func TestMessageCheck(t *testing.T) {
Cases: []Case{
{
"CLOSE must not fail",
Message{true, CLOSE, 125, []byte{0x03,0xe8,0}}, false, CloseFrame,
Message{true, CLOSE, 125, []byte{0x03, 0xe8, 0}}, false, CloseFrame,
}, {
"PING must not fail",
Message{true, PING, 125, []byte{}}, false, nil,
@ -372,7 +365,7 @@ func TestMessageCheck(t *testing.T) {
Cases: []Case{
{
"CLOSE must fail",
Message{true, CLOSE, 126, []byte{0x03,0xe8,0}}, false, ErrTooLongControlFrame,
Message{true, CLOSE, 126, []byte{0x03, 0xe8, 0}}, false, ErrTooLongControlFrame,
}, {
"PING must fail",
Message{true, PING, 126, []byte{}}, false, ErrTooLongControlFrame,
@ -386,7 +379,7 @@ func TestMessageCheck(t *testing.T) {
Cases: []Case{
{
"CLOSE must fail",
Message{false, CLOSE, 126, []byte{0x03,0xe8,0}}, false, ErrInvalidFragment,
Message{false, CLOSE, 126, []byte{0x03, 0xe8, 0}}, false, ErrInvalidFragment,
}, {
"PING must fail",
Message{false, PING, 126, []byte{}}, false, ErrInvalidFragment,
@ -411,23 +404,23 @@ func TestMessageCheck(t *testing.T) {
Cases: []Case{
{
"CLOSE valid reason",
Message{true, CLOSE, 5, []byte{0x03,0xe8, 0xe2,0x82,0xa1}}, false, CloseFrame,
Message{true, CLOSE, 5, []byte{0x03, 0xe8, 0xe2, 0x82, 0xa1}}, false, CloseFrame,
}, {
"CLOSE invalid reason byte 2",
Message{true, CLOSE, 5, []byte{0x03,0xe8, 0xe2,0x28,0xa1}}, false, ErrInvalidPayload,
Message{true, CLOSE, 5, []byte{0x03, 0xe8, 0xe2, 0x28, 0xa1}}, false, ErrInvalidPayload,
}, {
"CLOSE invalid reason byte 3",
Message{true, CLOSE, 5, []byte{0x03,0xe8, 0xe2,0x82,0x28}}, false, ErrInvalidPayload,
Message{true, CLOSE, 5, []byte{0x03, 0xe8, 0xe2, 0x82, 0x28}}, false, ErrInvalidPayload,
},
{
"TEXT valid reason",
Message{true, TEXT, 3, []byte{0xe2,0x82,0xa1}}, false, nil,
Message{true, TEXT, 3, []byte{0xe2, 0x82, 0xa1}}, false, nil,
}, {
"TEXT invalid reason byte 2",
Message{true, TEXT, 3, []byte{0xe2,0x28,0xa1}}, false, ErrInvalidPayload,
Message{true, TEXT, 3, []byte{0xe2, 0x28, 0xa1}}, false, ErrInvalidPayload,
}, {
"TEXT invalid reason byte 3",
Message{true, TEXT, 3, []byte{0xe2,0x82,0x28}}, false, ErrInvalidPayload,
Message{true, TEXT, 3, []byte{0xe2, 0x82, 0x28}}, false, ErrInvalidPayload,
},
},
}, {
@ -438,112 +431,111 @@ func TestMessageCheck(t *testing.T) {
Message{true, CLOSE, 1, []byte{0x03}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 1000",
Message{true, CLOSE, 2, []byte{0x03,0xe8}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xe8}}, false, CloseFrame,
}, {
"invalid CLOSE status 999 under 1000",
Message{true, CLOSE, 2, []byte{0x03,0xe7}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xe7}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 1001",
Message{true, CLOSE, 2, []byte{0x03,0xe9}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xe9}}, false, CloseFrame,
}, {
"valid CLOSE status 1002",
Message{true, CLOSE, 2, []byte{0x03,0xea}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xea}}, false, CloseFrame,
}, {
"valid CLOSE status 1003",
Message{true, CLOSE, 2, []byte{0x03,0xeb}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xeb}}, false, CloseFrame,
}, {
"invalid CLOSE status 1004",
Message{true, CLOSE, 2, []byte{0x03,0xec}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xec}}, false, ErrInvalidCloseStatus,
}, {
"invalid CLOSE status 1005",
Message{true, CLOSE, 2, []byte{0x03,0xed}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xed}}, false, ErrInvalidCloseStatus,
}, {
"invalid CLOSE status 1006",
Message{true, CLOSE, 2, []byte{0x03,0xee}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xee}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 1007",
Message{true, CLOSE, 2, []byte{0x03,0xef}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xef}}, false, CloseFrame,
}, {
"valid CLOSE status 1011",
Message{true, CLOSE, 2, []byte{0x03,0xf3}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xf3}}, false, CloseFrame,
}, {
"invalid CLOSE status 1012",
Message{true, CLOSE, 2, []byte{0x03,0xf4}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xf4}}, false, ErrInvalidCloseStatus,
}, {
"invalid CLOSE status 1013",
Message{true, CLOSE, 2, []byte{0x03,0xf5}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xf5}}, false, ErrInvalidCloseStatus,
}, {
"invalid CLOSE status 1014",
Message{true, CLOSE, 2, []byte{0x03,0xf6}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xf6}}, false, ErrInvalidCloseStatus,
}, {
"invalid CLOSE status 1015",
Message{true, CLOSE, 2, []byte{0x03,0xf7}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xf7}}, false, ErrInvalidCloseStatus,
}, {
"invalid CLOSE status 1016",
Message{true, CLOSE, 2, []byte{0x03,0xf8}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x03, 0xf8}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 1017",
Message{true, CLOSE, 2, []byte{0x03,0xf9}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x03, 0xf9}}, false, CloseFrame,
}, {
"valid CLOSE status 1099",
Message{true, CLOSE, 2, []byte{0x04,0x4b}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x04, 0x4b}}, false, CloseFrame,
}, {
"invalid CLOSE status 1100",
Message{true, CLOSE, 2, []byte{0x04,0x4c}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x04, 0x4c}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 1101",
Message{true, CLOSE, 2, []byte{0x04,0x4d}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x04, 0x4d}}, false, CloseFrame,
}, {
"valid CLOSE status 1999",
Message{true, CLOSE, 2, []byte{0x07,0xcf}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x07, 0xcf}}, false, CloseFrame,
}, {
"invalid CLOSE status 2000",
Message{true, CLOSE, 2, []byte{0x07,0xd0}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x07, 0xd0}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 2001",
Message{true, CLOSE, 2, []byte{0x07,0xd1}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x07, 0xd1}}, false, CloseFrame,
}, {
"valid CLOSE status 2998",
Message{true, CLOSE, 2, []byte{0x0b,0xb6}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x0b, 0xb6}}, false, CloseFrame,
}, {
"invalid CLOSE status 2999",
Message{true, CLOSE, 2, []byte{0x0b,0xb7}}, false, ErrInvalidCloseStatus,
Message{true, CLOSE, 2, []byte{0x0b, 0xb7}}, false, ErrInvalidCloseStatus,
}, {
"valid CLOSE status 3000",
Message{true, CLOSE, 2, []byte{0x0b,0xb8}}, false, CloseFrame,
Message{true, CLOSE, 2, []byte{0x0b, 0xb8}}, false, CloseFrame,
},
},
}, {
Name: "OpCode check",
Cases: []Case{
{ "0", Message{true, 0, 0, []byte{}}, false, ErrUnexpectedContinuation, },
{ "1", Message{true, 1, 0, []byte{}}, false, nil, },
{ "2", Message{true, 2, 0, []byte{}}, false, nil, },
{ "3", Message{true, 3, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "4", Message{true, 4, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "5", Message{true, 5, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "6", Message{true, 6, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "7", Message{true, 7, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "8", Message{true, 8, 0, []byte{}}, false, CloseFrame, },
{ "9", Message{true, 9, 0, []byte{}}, false, nil, },
{ "10", Message{true, 10, 0, []byte{}}, false, nil, },
{ "11", Message{true, 11, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "12", Message{true, 12, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "13", Message{true, 13, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "14", Message{true, 14, 0, []byte{}}, false, ErrInvalidOpCode, },
{ "15", Message{true, 15, 0, []byte{}}, false, ErrInvalidOpCode, },
{"0", Message{true, 0, 0, []byte{}}, false, ErrUnexpectedContinuation},
{"1", Message{true, 1, 0, []byte{}}, false, nil},
{"2", Message{true, 2, 0, []byte{}}, false, nil},
{"3", Message{true, 3, 0, []byte{}}, false, ErrInvalidOpCode},
{"4", Message{true, 4, 0, []byte{}}, false, ErrInvalidOpCode},
{"5", Message{true, 5, 0, []byte{}}, false, ErrInvalidOpCode},
{"6", Message{true, 6, 0, []byte{}}, false, ErrInvalidOpCode},
{"7", Message{true, 7, 0, []byte{}}, false, ErrInvalidOpCode},
{"8", Message{true, 8, 0, []byte{}}, false, CloseFrame},
{"9", Message{true, 9, 0, []byte{}}, false, nil},
{"10", Message{true, 10, 0, []byte{}}, false, nil},
{"11", Message{true, 11, 0, []byte{}}, false, ErrInvalidOpCode},
{"12", Message{true, 12, 0, []byte{}}, false, ErrInvalidOpCode},
{"13", Message{true, 13, 0, []byte{}}, false, ErrInvalidOpCode},
{"14", Message{true, 14, 0, []byte{}}, false, ErrInvalidOpCode},
{"15", Message{true, 15, 0, []byte{}}, false, ErrInvalidOpCode},
},
},
}
for _, tcc := range cases {
for _, tcc := range cases{
t.Run(tcc.Name, func(t *testing.T) {
t.Run(tcc.Name, func(t *testing.T){
for _, tc := range tcc.Cases {
for _, tc := range tcc.Cases{
t.Run(tc.Name, func(t *testing.T){
t.Run(tc.Name, func(t *testing.T) {
actual := tc.Msg.check(tc.WaitingFragment)

View File

@ -1,19 +1,18 @@
package ws
package websocket
import (
"net"
"fmt"
"git.xdrm.io/gws/internal/uri/parser"
"git.xdrm.io/go/websocket/internal/uri/parser"
"net"
)
// Represents all channels that need a server
type serverChannelSet struct{
type serverChannelSet struct {
register chan *client
unregister chan *client
broadcast chan Message
}
// Represents a websocket server
type Server struct {
sock net.Listener // listen socket
@ -27,11 +26,8 @@ type Server struct {
ch serverChannelSet
}
// CreateServer creates a server for a specific HOST and PORT
func CreateServer(host string, port uint16) *Server{
func CreateServer(host string, port uint16) *Server {
return &Server{
addr: []byte(host),
@ -53,11 +49,10 @@ func CreateServer(host string, port uint16) *Server{
}
// BindDefault binds a default controller
// it will be called if the URI does not
// match another controller
func (s *Server) BindDefault(f ControllerFunc){
func (s *Server) BindDefault(f ControllerFunc) {
s.ctl.Def = &Controller{
URI: nil,
@ -66,25 +61,25 @@ func (s *Server) BindDefault(f ControllerFunc){
}
// Bind binds a controller to an URI scheme
func (s *Server) Bind(uri string, f ControllerFunc) error {
/* (1) Build URI parser */
uriScheme, err := parser.Build(uri)
if err != nil { return fmt.Errorf("Cannot build URI: %s", err) }
if err != nil {
return fmt.Errorf("Cannot build URI: %s", err)
}
/* (2) Create controller */
s.ctl.Uri = append(s.ctl.Uri, &Controller{
URI: uriScheme,
Fun: f,
} )
})
return nil
}
// Launch launches the websocket server
func (s *Server) Launch() error {
@ -108,8 +103,6 @@ func (s *Server) Launch() error {
/* (3) Launch scheduler */
go s.scheduler()
/* (2) For each incoming connection (client)
---------------------------------------------------------*/
for {
@ -120,7 +113,7 @@ func (s *Server) Launch() error {
break
}
go func(){
go func() {
/* (2) Try to create client */
cli, err := buildClient(sock, s.ctl, s.ch)
@ -140,32 +133,31 @@ func (s *Server) Launch() error {
}
// Scheduler schedules clients registration and broadcast
func (s *Server) scheduler(){
func (s *Server) scheduler() {
for {
select {
/* (1) New client */
case client := <- s.ch.register:
case client := <-s.ch.register:
// fmt.Printf(" + client\n")
s.clients[client.io.sock] = client
/* (2) New client */
case client := <- s.ch.unregister:
case client := <-s.ch.unregister:
// fmt.Printf(" - client\n")
delete(s.clients, client.io.sock)
/* (3) Broadcast */
case msg := <- s.ch.broadcast:
case msg := <-s.ch.broadcast:
fmt.Printf(" + broadcast\n")
for _, c := range s.clients{
for _, c := range s.clients {
c.ch.send <- msg
}