Re: [Wireshark-dev] f5ethtrailer: disabled by default

2018-04-25 Thread Guy Harris
On Apr 25, 2018, at 10:37 AM, Jason Cohen wrote: > It's not a huge deal, but what would be necessary for the dissector to be > enabled by default? > > The commit message included: > Also disable F5ETHTRAILER by default since it doesn't have a discriminating > heuristic. It'd need a discriminat

Re: [Wireshark-dev] Overriding a builtin dissector

2018-04-25 Thread Roland Knall
And I think this would be the case here, with new formats, if nothing else changes Regards Roland > Am 25.04.2018 um 18:05 schrieb Graham Bloice : > > > >> On 25 April 2018 at 16:51, Pascal Quantin wrote: >> Hi Jason, >> >> 2018-04-25 17:40 GMT+02:00 Jason Cohen : >>> I've already asked som

[Wireshark-dev] f5ethtrailer: disabled by default

2018-04-25 Thread Jason Cohen
It's not a huge deal, but what would be necessary for the dissector to be enabled by default? The commit message included: Also disable F5ETHTRAILER by default since it doesn't have a discriminating heuristic. We could probably get the changes needed, but want make sure we understand the requirem

Re: [Wireshark-dev] Overriding a builtin dissector

2018-04-25 Thread Graham Bloice
On 25 April 2018 at 16:51, Pascal Quantin wrote: > Hi Jason, > > 2018-04-25 17:40 GMT+02:00 Jason Cohen : > >> I've already asked some of this privately, but submitting to the full >> list for archival as well. >> >> Is there a way or any thoughts of being able to override an existing, >> built-i

Re: [Wireshark-dev] Overriding a builtin dissector

2018-04-25 Thread Pascal Quantin
Hi Jason, 2018-04-25 17:40 GMT+02:00 Jason Cohen : > I've already asked some of this privately, but submitting to the full list > for archival as well. > > Is there a way or any thoughts of being able to override an existing, > built-in dissector? > You can deactivate a builtin dissector via the

[Wireshark-dev] Overriding a builtin dissector

2018-04-25 Thread Jason Cohen
I've already asked some of this privately, but submitting to the full list for archival as well. Is there a way or any thoughts of being able to override an existing, built-in dissector? The specific case in point being the f5ethtrailer dissector recently was included as a built-in. Would it be