Why map followed by foreach? In my head, this should work:
```scala
subscriptions.valuesIterator.foreach({
  case Right(subscription) => subscription !? SubscriptionProtocol.Kill
})
```

Also, what would happen on a Left value?