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

Implement auth #322

Closed
wants to merge 4 commits into from
Closed

Implement auth #322

wants to merge 4 commits into from

Conversation

sverben
Copy link
Contributor

@sverben sverben commented Aug 4, 2022

This enables online mode using a chrome extension
Extension

Possible enhancements:

  • Add a button on the play_screen component to display where to download the extension
  • Upload to Chrome Web Store

@sverben
Copy link
Contributor Author

sverben commented Aug 4, 2022

This closes #251 and #27

@rom1504
Copy link
Member

rom1504 commented Aug 7, 2022

interesting solution.
Could someone try it ? maybe @AwesomestCode ?

@IlanLuci
Copy link

Doesn't seem to be working on 1.8.9 and potentially other versions as well. When I try to join minemen.club or hypixel i get "RangeError: Invalid WebSocket frame: RSV1 must be clear" referencing events.js:377
throw er; // Unhandled 'error' event -- not sure if this is something on my end or a problem for others as well. I'm on a mac testing on brave browser. I am going to install chrome rn and see if its a problem there too.

@Verdanix
Copy link

Verdanix commented Nov 3, 2022

When will this get approved? If it even will...

@rom1504
Copy link
Member

rom1504 commented Mar 4, 2023

seems it doesn't work so let's not merge

@rom1504 rom1504 closed this Mar 4, 2023
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