Multiauth

[DEATHB4DEFEAT]

No description provided.

[Copilot]

Copilot reviewed 52 out of 52 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

Robust.Client/Graphics/Clyde/Clyde.Rendering.cs:420

• When disabling the scissor test, a call to CheckGlError() is omitted. For consistent error checking and debugging, consider invoking CheckGlError() after disabling the state.

else { GL.Disable(EnableCap.ScissorTest); }

[Copilot]

Hardcoding the column index when retrieving the serverUrl makes the code fragile if the query order ever changes. Consider using named columns or defined constants to ensure the correct value is extracted even if the SQL query is updated.

Suggested change

	var userId = Guid.Parse(reader.GetString(0));
	var userName = reader.GetString(1);
	var token = reader.GetString(2);
	serverUrl = reader.GetString(4);
	var userId = Guid.Parse(reader.GetString(reader.GetOrdinal("UserId")));
	var userName = reader.GetString(reader.GetOrdinal("UserName"));
	var token = reader.GetString(reader.GetOrdinal("Token"));
	serverUrl = reader.GetString(reader.GetOrdinal("ServerUrl"));