search

kdcllc / CometD.NetCore

CometD for use with Salesforce Platform Events
MIT License
29 stars 16 forks source link

AbstractClientSession Thread-Safe #9

Closed brunoadamalves closed 4 years ago

brunoadamalves commented 4 years ago

Hi guys, Just to double-check the class AbstractClientSession, line 17, the list _extensions, any concern about using ArrayList.Synchronized?

kdcllc commented 4 years ago

@brunoadamalves yes this should be thread safe. I am including https://jonlabelle.com/snippets/view/csharp/generic-thread-safe-list with the next release