Skip to content
Snippets Groups Projects
Select Git revision
  • master default protected
  • release/202005
  • release/202001
  • release/201912
  • release/windows-test/201910
  • release/201908
  • release/201906
  • release/201905
  • release/201904
  • release/201903
  • release/201902
  • release/201901
  • release/201812
  • release/201811
  • release/201808
  • wip/localPushNotifications
  • wip/newAccountService
  • wip/rxNameService
  • wip/rxAccountsService
19 results
Blame Permalink
  • Andreas Traczyk's avatar
    fd2c3787
    notifications: stream HTTP data instead of waiting for response · fd2c3787
    Andreas Traczyk authored and Kateryna Kostiuk's avatar Kateryna Kostiuk committed 
    The proxy-server route /{key} produces a HTTP stream which reflects values obtained by a DHT get. The stream is not guaranteed to signal termination. Previously we were waiting on a complete response which in several scenarios would never arrive. This causes the stream data to not be parsed, and for the URLSession block until a timeout, which would cause queueing of any incoming notifications.

This commit changes the design to stream the HTTP content and handle lines (JSON) asynchronously as they are received.

Gitlab: #359
Change-Id: I469bdde0a3c1ad9d0ac46285740c0f55fd377018
    fd2c3787
    History
    notifications: stream HTTP data instead of waiting for response
    Andreas Traczyk authored and Kateryna Kostiuk's avatar Kateryna Kostiuk committed 
    The proxy-server route /{key} produces a HTTP stream which reflects values obtained by a DHT get. The stream is not guaranteed to signal termination. Previously we were waiting on a complete response which in several scenarios would never arrive. This causes the stream data to not be parsed, and for the URLSession block until a timeout, which would cause queueing of any incoming notifications.

This commit changes the design to stream the HTTP content and handle lines (JSON) asynchronously as they are received.

Gitlab: #359
Change-Id: I469bdde0a3c1ad9d0ac46285740c0f55fd377018
Code owners
Assign users and groups as approvers for specific file changes. Learn more.