[GitHub] [trafficserver] maskit commented on issue #9428: Prohibit a second SETTINGS frame

["maskit (via GitHub)" <gi...@apache.org>]

maskit commented on issue #9428:
URL: https://github.com/apache/trafficserver/issues/9428#issuecomment-1593710273

   > I noticed that we don't seem to enforce one control stream per connection?
   
   There are many such kind of things. I could file more issues on the project but it'd be overwhelming.
   
   > Also maybe we should rename _control_stream_dispatcher to something else, since we use it to handle PUSH streams too?
   
   That is an option. We could also introduce _push_stream_dispatcher, but I'm not too sure if it's a good way of handling those. As you can see, current design is extensible for future extension frames but at the same time it's not great if many frame handlers have to be registered to one dispatcher. I'd say it's nice to have three dispatchers.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org