Skip to content

Conversation

@JerrySB03
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings May 13, 2025 11:56
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the client to use a standardized Id property, adds a Name field, strengthens RSA-based connection setup, and introduces basic in-memory and SQLite-backed user registries.

  • Renames id to Id, makes RsaPub public, and adds Name on Client.
  • Refactors EstablishConnection/RecieveConnection to exchange an RSA-encrypted key.
  • Adds a simple Database class and updates the EF model and SQLite context to store usernames.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
sutor-aes/Client.cs Renamed properties, added Name, updated RSA key exchange logic, and appended an in-memory Database class.
Models/DBmodels.cs Added Name property with unique index on DBUser.
Database/SQLite.cs Updated RegisterUser to accept username and set the new Name field.
Comments suppressed due to low confidence (1)

sutor-aes/Client.cs:12

  • [nitpick] The public property RsaPub is abbreviated; for clarity and consistency, consider renaming it to RsaPublicKey.
public byte[] RsaPub { get; set; }

@JerrySB03 JerrySB03 merged commit 9102d53 into main May 13, 2025
1 check passed
@JerrySB03 JerrySB03 deleted the feat/client branch May 13, 2025 12:21
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.

3 participants