Github user mmiklavc commented on a diff in the pull request:
https://github.com/apache/metron/pull/1154#discussion_r208673702
--- Diff:
metron-interface/metron-alerts/src/app/pcap/service/pcap.service.ts ---
@@ -56,8 +56,7 @@ export class PcapService {
public getPackets(id: string, pageId: number): Observable<Pdml> {
return this.http.get(`/api/v1/pcap/${id}/pdml?page=${pageId}`, new
RequestOptions({headers: new Headers(this.defaultHeaders)}))
.map(HttpUtil.extractData)
- .catch(HttpUtil.handleError)
- .onErrorResumeNext();
--- End diff --
What are the implications of removing `onErrorResumeNext()`?
---
