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 authentication #22

Merged
merged 4 commits into from
Dec 12, 2020
Merged

Update authentication #22

merged 4 commits into from
Dec 12, 2020

Conversation

AlexWayfer
Copy link
Collaborator

@AlexWayfer AlexWayfer commented Nov 4, 2020

  • Require client_id and client_secret for Application Access Token.
  • Use twitch_oauth2 gem for authentication.
  • Drop with_raw option, include it always.
  • Drop webmock development dependency, use faraday interface for VCR.
  • Use configure_rspec_metadata! for auto-generated cassettes names.
  • Split Client's methods into modules.

@AlexWayfer
Copy link
Collaborator Author

@mauricew, are you OK with these changes?

I've used twitch_oauth2 gem for twitch-rb, and it's common between Twitch API Kraken and Helix versions.

Copy link
Owner

@mauricew mauricew left a comment

Choose a reason for hiding this comment

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

Looks good, just have a few small doc changes and a possible added option for making token refresh opt-in.

README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
lib/twitch/client.rb Show resolved Hide resolved
lib/twitch/client.rb Outdated Show resolved Hide resolved
@AlexWayfer AlexWayfer force-pushed the update_authentication branch from 30bb372 to d910ddc Compare November 13, 2020 16:16
*   Require `client_id` and `client_secret` for Application Access Token.
*   Use `twitch_oauth2` gem for authentication.
*   Drop `with_raw` option, include it always.
*   Drop `webmock` development dependency, use `faraday` interface for VCR.
*   Use `configure_rspec_metadata!` for auto-generated cassettes names.
*   Split `Client`'s methods into modules.
@AlexWayfer AlexWayfer force-pushed the update_authentication branch from a3dbe3d to 5d5849c Compare November 17, 2020 00:27
@AlexWayfer AlexWayfer mentioned this pull request Nov 19, 2020
@AlexWayfer AlexWayfer changed the title Update authentication WIP: Update authentication Nov 19, 2020
@AlexWayfer AlexWayfer requested a review from mauricew November 30, 2020 15:50
@AlexWayfer
Copy link
Collaborator Author

AlexWayfer requested a review from mauricew yesterday

This was wrong, I forgot about "WIP" and its reason, sorry.

AlexWayfer added a commit to AlexWayfer/twitch_oauth2 that referenced this pull request Dec 4, 2020
@AlexWayfer AlexWayfer changed the title WIP: Update authentication Update authentication Dec 4, 2020
Copy link
Owner

@mauricew mauricew left a comment

Choose a reason for hiding this comment

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

Everything looks good!

@mauricew mauricew merged commit afdea35 into master Dec 12, 2020
@mauricew mauricew deleted the update_authentication branch December 12, 2020 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants