Skip to content

[SR-10428] URLSession: Methods invalidateAndCancel and getAllTasks implemented #1934

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

karthikkeyan
Copy link
Contributor

@karthikkeyan karthikkeyan commented Feb 21, 2019

  • invalidateAndCancel method implemented.
  • getAllTasks methods implemented.
  • New computed property allTasks added in URLSession._TaskRegistry, to return an array of all the existing tasks added in _TaskRegistry.

Update:
Bug Link: https://bugs.swift.org/browse/SR-10428

@spevans
Copy link
Contributor

spevans commented Feb 21, 2019

@swift-ci test

@karthikkeyan
Copy link
Contributor Author

@spevans I just pushed one more commit, with minor code change. Please do a CI test one more time, If it necessary.

@spevans
Copy link
Contributor

spevans commented Feb 22, 2019

@swift-ci test

- invalidateAndCancel method implemented.
- getAllTasks methods implemented.
- New computed property allTasks added in URLSession._TaskRegistry, to return an array of all the existing tasks added in _TaskRegistry.
…terInvalidateAndCancel unit test from TestURLSession file.
@karthikkeyan karthikkeyan force-pushed the karthik/session-invalidate-cancel branch from 22de662 to c9d1eda Compare April 9, 2019 19:47
@karthikkeyan karthikkeyan changed the title URLSession: Methods invalidateAndCancel and getAllTasks implemented [SR-10428] URLSession: Methods invalidateAndCancel and getAllTasks implemented Apr 9, 2019
@karthikkeyan
Copy link
Contributor Author

I have fixed the merge conflict and updated the PR with an appropriate ticket number

@spevans
Copy link
Contributor

spevans commented Apr 9, 2019

@swift-ci test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants