svg-to-geojson icon indicating copy to clipboard operation
svg-to-geojson copied to clipboard

Possible race condition issue in `streamProps`

Open fahad19 opened this issue 7 years ago • 0 comments

Current situation

We ship streamProps helper function from frint-component-utils package, which allows combining values from various sources and then generate a single observable of props.

It uses concatMap operator from RxJS internally.

Possible bug

There is a possibility that with concatMap, we will have some race condition issue.

It is advised to use switchMap instead.

What to do

  • Write unit tests proving there is a race condition issue with streamProps
  • If there is, replace concatMap usage with switchMap without breaking the API

fahad19 avatar Nov 28 '17 22:11 fahad19