Skip to content

feat: Bool to skip password and auth check in RemoteStatistics.cs - #4001

Merged
miwarnec merged 2 commits into
masterfrom
RemoteStatisticsSkipPassBool
Apr 16, 2025
Merged

feat: Bool to skip password and auth check in RemoteStatistics.cs#4001
miwarnec merged 2 commits into
masterfrom
RemoteStatisticsSkipPassBool

Conversation

@JesusLuvsYooh

Copy link
Copy Markdown
Contributor

Allows for quicker testing of remote statistics, or when clients viewing the data is just not a threat.
It is false by default.

Screenshot 2025-03-23 at 10 14 35

@miwarnec

Copy link
Copy Markdown
Collaborator

@JesusLuvsYooh this is fine, but can we avoid the double negative?
RequiresPasswordAuth=true by default would be better, otherwise makes my brain hurt

@JesusLuvsYooh

Copy link
Copy Markdown
Contributor Author

RequiresPasswordAuth

Done :) updated variable name

@miwarnec
miwarnec merged commit a2fb3cc into master Apr 16, 2025
@miwarnec
miwarnec deleted the RemoteStatisticsSkipPassBool branch April 16, 2025 08:06
MrGadget1024 pushed a commit that referenced this pull request Apr 17, 2025
)

* Update RemoteStatistics.cs

* Updated variable name, and added tooltip/comments.
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.

3 participants