Skip to content

Commit 83c78e7

Browse files
author
Daniel Herzog
committed
Workaround CORE-47687: UrlFinished is missing for urls resulting in a 301 (Moved Permanently) response
1 parent 36b27a4 commit 83c78e7

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

src/network/network_service.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -590,6 +590,7 @@ cls.NetworkLoggerEntry = function(id, resource_id, document_id, context_starttim
590590
this.called_get_body = false;
591591
this.current_response = null;
592592
this.current_request = null;
593+
this.set_is_finished_on_responsefinished = false;
593594
};
594595

595596
cls.NetworkLoggerEntryPrototype = function()
@@ -878,6 +879,9 @@ cls.NetworkLoggerEntryPrototype = function()
878879
if (event.data && event.data.mimeType)
879880
this.mime = event.data && event.data.mimeType;
880881

882+
if (this.set_is_finished_on_responsefinished)
883+
this.is_finished = true;
884+
881885
this._guess_response_type();
882886
};
883887

@@ -894,7 +898,8 @@ cls.NetworkLoggerEntryPrototype = function()
894898

895899
this._update_event_urlredirect = function(event)
896900
{
897-
// This does not add any information, the event is only used to change the requestID
901+
// Workaround for CORE-47687
902+
this.set_is_finished_on_responsefinished = true;
898903
};
899904

900905
this._guess_response_type = function()

0 commit comments

Comments
 (0)