Browse Source

Updated TODO

Brad Fitzpatrick 11 years ago
parent
commit
c57de9fa40
1 changed files with 7 additions and 2 deletions
  1. 7 2
      server.go

+ 7 - 2
server.go

@@ -1012,8 +1012,13 @@ func (sc *serverConn) processData(f *DataFrame) error {
 		return StreamError{id, ErrCodeStreamClosed}
 		return StreamError{id, ErrCodeStreamClosed}
 	}
 	}
 	if len(data) > 0 {
 	if len(data) > 0 {
-		// TODO: verify they're allowed to write with the flow control
-		// window we'd advertised to them.
+		// TODO: verify they're allowed to write with the flow
+		// control window we'd advertised to them. (currently
+		// this is fails elsewhere, in that the body buffer is
+		// always 65k, the default initial window size, but
+		// once that's fixed to grow and shrink on demand,
+		// we'll need to be stricter before that, or in the
+		// buffer code)
 		wrote, err := st.body.Write(data)
 		wrote, err := st.body.Write(data)
 		if err != nil {
 		if err != nil {
 			return StreamError{id, ErrCodeStreamClosed}
 			return StreamError{id, ErrCodeStreamClosed}