poco icon indicating copy to clipboard operation
poco copied to clipboard

Net: fixed parsing of media type in message header

Open vm2mv opened this issue 8 years ago • 3 comments

vm2mv avatar Nov 21 '16 08:11 vm2mv

What issue does this solve? According to the RFC, a semicolon is the only allowed separator.

obiltschnig avatar Nov 22 '16 09:11 obiltschnig

Our software processes lots of data received from different sources. It processes requests from many programs web browsers as well as responses from different servers. Unfortunately we often see data where comma (',') is used as separator

vm2mv avatar Nov 22 '16 11:11 vm2mv

OK, I understand your point. Will have to think about it. My issue with such fixes is that the bad code sending badly formatted messages never gets fixed if everyone else changes their code to accept these badly formatted messages. It may be the right thing to do for your application, but the question is whether it's the right thing to do for a general purpose library like POCO.

obiltschnig avatar Nov 22 '16 13:11 obiltschnig