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

Update oauth2 crate to v3 #2600

Merged
merged 1 commit into from Jul 5, 2020
Merged

Update oauth2 crate to v3 #2600

merged 1 commit into from Jul 5, 2020

Conversation

JohnTitor
Copy link
Member

Release notes: https://github.com/ramosbugs/oauth2-rs/releases

This pulls two versions of http crate, it should be resolved once ramosbugs/oauth2-rs#103 is shipped. Otherwise, we can drop some (duplicate) dependencies. Also updates url crate to v2.

r? @jtgeibel

tar = "0.4.16"
base64 = "0.12"

openssl = "0.10.13"
oauth2 = "2.0.0"
oauth2 = { version = "3.0.0", default-features = false, features = ["reqwest-010"] }
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a hack until ramosbugs/oauth2-rs#102 is shipped.

@jtgeibel
Copy link
Member

jtgeibel commented Jul 5, 2020

Looks great to me! Login worked fine for me locally, will also test in staging.

@bors r+

@bors
Copy link
Contributor

bors commented Jul 5, 2020

📌 Commit 3d9e698 has been approved by jtgeibel

@bors
Copy link
Contributor

bors commented Jul 5, 2020

⌛ Testing commit 3d9e698 with merge 4c648db...

@bors
Copy link
Contributor

bors commented Jul 5, 2020

☀️ Test successful - checks-travis
Approved by: jtgeibel
Pushing 4c648db to master...

@bors bors merged commit 4c648db into rust-lang:master Jul 5, 2020
@JohnTitor JohnTitor deleted the oauth2 branch July 5, 2020 22:22
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.

None yet

4 participants