to packet server

Signed-off-by: Daniel Mangum <[email protected]>
pion · Jul 18, 2023 · 51bba14 · 51bba14
1 parent 9a6591a
commit 51bba14
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 11 deletions.
diff --git a/conn.go b/conn.go
@@ -268,7 +268,10 @@ func Server(conn net.Conn, config *Config) (*Conn, error) {
 	return ServerWithContext(ctx, conn, config)
 }
 
-func UDPServer(ctx context.Context, conn net.PacketConn, rAddr net.Addr, config *Config) (*Conn, error) {
+// PacketServer listens for incoming DTLS connections.
+// Unlike Server, PacketServer allows for connections to change remote address.
+// The provided rAddr will be used as the initial remote address for sending.
+func PacketServer(ctx context.Context, conn net.PacketConn, rAddr net.Addr, config *Config) (*Conn, error) {
 	return createConn(ctx, conn, rAddr, config, true, nil)
 }
 

diff --git a/pkg/protocol/recordlayer/inner_plaintext.go b/pkg/protocol/recordlayer/inner_plaintext.go
@@ -38,7 +38,6 @@ func (p *InnerPlaintext) Unmarshal(data []byte) error {
 		i--
 	}
 	if i == 0 {
-		// TODO: more specific error for missing content type
 		return errBufferTooSmall
 	}
 	p.RealType = protocol.ContentType(data[i])

diff --git a/pkg/protocol/recordlayer/recordlayer.go b/pkg/protocol/recordlayer/recordlayer.go
@@ -13,15 +13,15 @@ import (
 
 // DTLS fixed size record layer header when Connection IDs are not in-use.
 
-// --------------------------------
-// | Type   |   Version   | Epoch |
-// --------------------------------
-// | Epoch  |    Sequence Number  |
-// --------------------------------
-// |   Sequence Number   | Length |
-// --------------------------------
-// | Length |      Fragment...    |
-// --------------------------------
+// ---------------------------------
+// | Type   |   Version   |  Epoch |
+// ---------------------------------
+// | Epoch  |    Sequence Number   |
+// ---------------------------------
+// |   Sequence Number   |  Length |
+// ---------------------------------
+// | Length |      Fragment...     |
+// ---------------------------------
 
 // fixedHeaderLenIdx is the index at which the record layer content length is
 // specified in a fixed length header (i.e. one that does not include a