Added feedback for trace deletions.

If you are on the page for a trace that got deleted, you will be redirected to the langing page.

Author: dylemma

codepulse/src/main/resources/toserve/pages/traces/trace-recording-controls.js

@@ -537,6 +537,9 @@
 				endTraceButton.overlay('ready')
 				traceRunningBus.push(false)
 				break
+			case 'deleted':
+				alert('This trace has been deleted. You will be redirected to the home screen')
+				window.location.href = '/'
 			}
 		})
 

codepulse/src/main/scala/com/secdec/codepulse/tracer/AkkaTracingTarget.scala

@@ -32,13 +32,16 @@ import reactive.EventSource
 import reactive.EventStream
 import com.secdec.codepulse.data.trace.TraceId
 import com.secdec.codepulse.data.trace.TraceData
+import akka.actor.PoisonPill
 
 sealed trait TracingTargetEvent
 object TracingTargetEvent {
 	case object Connecting extends TracingTargetEvent
 	case object Connected extends TracingTargetEvent
 	case class Started(traceId: TraceId) extends TracingTargetEvent
 	case class Finished(reason: TraceEndReason) extends TracingTargetEvent
+
+	case object Deleted extends TracingTargetEvent
 }
 
 sealed trait TracingTargetState
@@ -70,6 +73,8 @@ trait TracingTarget {
 
 	def traceData: TraceData
 	def transientData: TransientTraceData
+
+	def requestDeletion(): Unit
 }
 
 object AkkaTracingTarget {
@@ -86,6 +91,8 @@ object AkkaTracingTarget {
 	private case class TraceConnected(trace: Trace)
 	private case class TraceEnded(reason: TraceEndReason)
 
+	private case object RequestDeletion
+
 	// implicit Timeout used for the Akka ask (?) method in getState
 	private implicit val timeout = new Timeout(5000)
 
@@ -97,6 +104,7 @@ object AkkaTracingTarget {
 			val reply = actor ? RequestState
 			reply.mapTo[TracingTargetState]
 		}
+		def requestDeletion() = actor ! RequestDeletion
 	}
 
 	def apply(actorSystem: ActorSystem, traceId: TraceId, traceData: TraceData, transientTraceData: TransientTraceData): TracingTarget = {
@@ -152,6 +160,9 @@ class AkkaTracingTarget(traceId: TraceId, traceData: TraceData, transientTraceDa
 		body orElse {
 			case sub: Subscribe => sub.f(events)
 			case RequestState => sender ! s
+			case RequestDeletion =>
+				events.fire(TracingTargetEvent.Deleted)
+				self ! PoisonPill
 		}
 	}
 

codepulse/src/main/scala/com/secdec/codepulse/tracer/TraceManager.scala

@@ -90,6 +90,7 @@ class TraceManager(val actorSystem: ActorSystem) extends Observing {
 		for (trace <- traces remove traceId) yield {
 			dataProvider.removeTrace(traceId)
 			traceListUpdates fire ()
+			trace.requestDeletion()
 			trace
 		}
 	}

codepulse/src/main/scala/com/secdec/codepulse/tracer/snippet/CometTracerUI.scala

@@ -66,6 +66,8 @@ class CometTracerUI extends CometWidget[TracingTarget, CometTracerUI]
 						case TraceEndReason.Halted => "halted"
 					}
 					sendStatusUpdate { Map("state" -> "finished", "reason" -> s) }
+				case TracingTargetEvent.Deleted =>
+					sendStatusUpdate { Map("state" -> "deleted") }
 			}
 		}
 	}