Skip to content

Add authenticate_user method (v2.0.4)#197

Open
Keith-wright wants to merge 7 commits intomasterfrom
feat/authenticate-user-2.0.4
Open

Add authenticate_user method (v2.0.4)#197
Keith-wright wants to merge 7 commits intomasterfrom
feat/authenticate-user-2.0.4

Conversation

@Keith-wright
Copy link

Summary

Supersedes #191. Adds the authenticate_user method for the user authentication flow, which was missing from the Ruby SDK while already present in the Node and PHP SDKs.

Changes

  • Added authenticate_user(socket_id, user_data) to Pusher::Client
  • Extracted shared auth logic into Pusher::Utils module
  • Fixed authentication_string in Pusher::Channel to correctly validate the custom_string argument type
  • Updated documentation link for authenticate method
  • Added tests for authenticate_user

Supersedes

#191

Copy link
Member

@aonemd aonemd left a comment

Choose a reason for hiding this comment

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

Tests are not running. Could you update the GH workflows

ubuntu-20.04 runners were retired from GitHub Actions, causing jobs to
be permanently stuck in queued state. Ruby 2.6, 2.7, and 3.0 are also
EOL. Update to ubuntu-latest and currently supported Ruby versions.
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.

4 participants