Skip to content
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

Convert to Async Await, Add configurable environments, remove deprecated code #130

Merged
merged 24 commits into from
Jun 10, 2022

Conversation

kylebrowning
Copy link
Member

@kylebrowning kylebrowning commented Jun 9, 2022

This PR does the following.

  • Removes the custom NIO code in favor of AsyncHttpClient
  • removes deprecated code
  • Allows environment to be sent on configure
  • Moves The bearer factory to be an actor
  • removes JWTKit in favor of Swift-crypto and a smaller EDSA signature

These are all breaking changes and as such will require a major version bump

@kylebrowning
Copy link
Member Author

@lgaches, @TimAEllis, @LukasCZ any interest in testing I didn't break anything for you?

@kylebrowning kylebrowning requested review from 0xTim and fabianfett June 9, 2022 23:50
@lgaches
Copy link
Contributor

lgaches commented Jun 10, 2022

Sure, I can test. Would be nice to update also the README file sample code.

@kylebrowning
Copy link
Member Author

Will definitely update the readme!

@lgaches
Copy link
Contributor

lgaches commented Jun 10, 2022

It's working for me

Package.swift Outdated Show resolved Hide resolved
@kylebrowning
Copy link
Member Author

I made just a few more changes. but mostly just hiding things from the public namespce

@kylebrowning kylebrowning merged commit 2180617 into master Jun 10, 2022
@kylebrowning kylebrowning deleted the kb/async-await branch June 14, 2022 16:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-major-version-bump This PR requires a major version bump
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants