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

feat: adding is authenticated bool to network player #828

Merged
merged 1 commit into from
Jun 1, 2021

Conversation

James-Frowen
Copy link
Member

Adding an explicit way to check if a network authenticator has accepted a client. some Authenticators dont set AuthenticationData so without the bool it is impossible to check if a player is authenticated or not.

Adding an explicit way to check if a network authenticator has accepted a client. some Authenticators dont set `AuthenticationData` so without the bool it is impossible to check if a player is authenticated or not.
@sonarcloud
Copy link

sonarcloud bot commented Jun 1, 2021

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@James-Frowen James-Frowen marked this pull request as ready for review June 1, 2021 20:29
@James-Frowen James-Frowen merged commit 372fd70 into master Jun 1, 2021
@James-Frowen James-Frowen deleted the adding-auth-bool branch June 1, 2021 23:55
github-actions bot pushed a commit that referenced this pull request Jun 2, 2021
# [96.1.0](v96.0.0...v96.1.0) (2021-06-02)

### Bug Fixes

* moving syncvar sender to networkserver so it gets intilized earlier ([8b2b828](8b2b828))
* stopping Server.Stop being called twice ([b950d39](b950d39))

### Features

* adding is authenticated bool to network player ([#828](#828)) ([372fd70](372fd70))
@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2021

🎉 This PR is included in version 96.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant