Skip to content

allows HTTPNetworkTransport to take URLSession instance#2

Merged
GevaZeichner merged 1 commit intoButterflyNetwork:intercept-requestfrom
chrislconover:session_delegate_support
Jul 18, 2019
Merged

allows HTTPNetworkTransport to take URLSession instance#2
GevaZeichner merged 1 commit intoButterflyNetwork:intercept-requestfrom
chrislconover:session_delegate_support

Conversation

@GevaZeichner
Copy link
Copy Markdown

@GevaZeichner GevaZeichner commented Jul 17, 2019

This brings apollographql#265 into our fork.
We need this capability so we can handle the session's URLAuthenticationChallenge.

…caller to handle exceptional cases by way of URLSessionDelegate
@GevaZeichner GevaZeichner requested a review from vineetshah July 17, 2019 22:57
@taion
Copy link
Copy Markdown

taion commented Jul 18, 2019

might be worth updating this fork at some point. it looks a decent bit behind. though i guess you may not be able to upgrade to the versions that use swift 5 yet?

@GevaZeichner
Copy link
Copy Markdown
Author

might be worth updating this fork at some point. it looks a decent bit behind. though i guess you may not be able to upgrade to the versions that use swift 5 yet?

I think it was last updated in May. I'll put it in our todos. We switched to Swift 5 so it shouldn't be a blocker. Any significant new features you know of?

@GevaZeichner GevaZeichner merged commit b386f7a into ButterflyNetwork:intercept-request Jul 18, 2019
@taion
Copy link
Copy Markdown

taion commented Jul 18, 2019

it looks like there were some interesting refactors in v0.12: https://github.com/apollographql/apollo-ios/blob/master/CHANGELOG.md#v0120

though at least on the cloud side, for upgrades, it's more like "why not"

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