[reSIProcate] removing AssumePath, WebRTC
Hi Scott,
I noticed your commit[1] removing AssumePath from repro.config
I previously set this to true for WebRTC use cases, I don't recall why,
it may have been based on the description of this option in repro.config
Looking at the history, it was added[2] in 2010 with this note:
* Addition of --assume-path option. For repro outbound support, this
option makes repro act as though the Supported header includes the
'path'
token. This is useful when trying to fake outbound support within
older
clients.
and repro/ReponseContext.cxx was modified[3] to stop using it in 2011.
I need to remove this in the sample repro.config for Debian, maybe it is
in other examples for WebRTC elsewhere, I'll check.
WebRTC clients probably add the "path" parameter or can be easily
patched to do so. What's more, the logic behind this was actually
remove in 2011 before we added WebRTC support in 2013. Looking at the
remaining code that you removed, it is not clear how setting this option
would have made any difference to WebRTC users anyway.
Is there anything else we should do in repro.config to make it obvious
how WebRTC users can get up and running quickly? I tried to add the
string "WebRTC" to the comments for every config option that impacts WebRTC
Regards,
Daniel
1.
https://github.com/resiprocate/resiprocate/commit/4ae009e93d94690cde60ea1a504db492e4c863b2
2.
https://github.com/resiprocate/resiprocate/commit/c23b86e67cd679bb991bac7447c1d9197ff921ac
3.
https://github.com/resiprocate/resiprocate/commit/4372bb9e09f75dcc2de72f628def60de1e9f24f4