1 by Den1s-coder · Pull Request #98 · Den1s-coder/MySocialNetwork

Den1s-coder · 2025-12-06T16:57:17Z

No description provided.

…ve it

…itory

Copilot

Pull request overview

This is a comprehensive feature update adding JWT refresh token authentication, Google OAuth, real-time notifications via SignalR, friendship management, reactions system, user settings, and pagination. The PR reorganizes domain entities into namespaces, migrates from single access tokens to access/refresh token pairs, adds new UI components including notification bells and avatar displays, and includes significant mobile app improvements.

Changes:

Authentication system upgraded with refresh tokens and Google OAuth integration
Real-time notification system added using SignalR with MediatR event dispatching
Friendship system and reactions (posts/comments/messages) implemented
Frontend reorganized with new Settings page, pagination, and improved UI components

Reviewed changes

Copilot reviewed 157 out of 161 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
backend/SocialNetwork.Domain/Entities/*	Reorganized entities into namespaces (Users, Posts, Comments, Chats, Reactions)
backend/SocialNetwork.Application/Events/*	Added MediatR event handlers for post/comment creation notifications
backend/SocialNetwork.API/Controllers/AuthController.cs	Added refresh token and Google OAuth endpoints
backend/SocialNetwork.API/Hubs/NotificationHub.cs	New SignalR hub for real-time notifications
frontend/socialnetwork.client/src/hooks/authFetch.js	New hook handling automatic token refresh on 401
frontend/socialnetwork.client/src/pages/Settings.jsx	New settings page for password/email changes
frontend/socialnetwork.client/src/components/NotificationBell.jsx	Real-time notification UI component
frontend/SocialNetwork.mobile/*	Mobile app updates for improved chat UI and authentication

Files not reviewed (3)

backend/SocialNetwork.Infrastructure/Migrations/20251225105208_AddRefreshToken.Designer.cs: Language not supported
backend/SocialNetwork.Infrastructure/Migrations/20260127124428_AddNotifications.Designer.cs: Language not supported
backend/SocialNetwork.Infrastructure/Migrations/20260216130431_AddReactions.Designer.cs: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-20T12:27:46Z

backend/SocialNetwork.Application/Interfaces/IMessageService.cs

    {
-        public Task<IEnumerable<Message>> GetMessageByChatIdAsync(Guid chatid);
+        public Task<IEnumerable<MessageDto>> GetMessageByChatIdAsync(Guid chatid, CancellationToken cancellationToken = default);
+        public Task ToogleReactionAsync(Guid messageId, Guid userId, Guid reactionType, CancellationToken cancellationToken = default);


Spelling error in method name: "ToogleReactionAsync" should be "ToggleReactionAsync" (double 'g'). This typo is consistent in the interface, service implementation, and controller usage.

Copilot · 2026-02-20T12:27:47Z

frontend/SocialNetwork.mobile/SocialNetwork.mobile/Services/ChatServiceClient.cs

+                    options.HttpMessageHandlerFactory = (messageHandler) =>
+                    {
+                        if (messageHandler is HttpClientHandler clientHandler)
+                        {
+                            clientHandler.ServerCertificateCustomValidationCallback = (sender, cert, chain, sslPolicyErrors) => true;
+                            return clientHandler;
+                        }
+
+                        return new HttpClientHandler
+                        {
+                            ServerCertificateCustomValidationCallback = (sender, cert, chain, sslPolicyErrors) => true
+                        };
+                    };


Security concern: SSL certificate validation is completely disabled for SignalR connections. This makes the connection vulnerable to man-in-the-middle attacks. Remove this override for production or at minimum add environment-based configuration to only disable it in development.

Copilot · 2026-02-20T12:27:47Z

backend/SocialNetwork.Application/Service/FriendService.cs

+            var AreFriends = await _friendshipRepository.AreFriendsAsync(UserId, friendId);
+            if (AreFriends)
+                throw new Exception("Users are not friends");


Logic error: The condition checks if AreFriends is true and then throws "Users are not friends". This should be if (!AreFriends) to throw when users are NOT friends.

Copilot · 2026-02-20T12:27:47Z

backend/SocialNetwork.Application/Service/FriendService.cs

+        public Task AcceptFriendRequest(Guid requestId, CancellationToken cancellationToken = default)
+        {
+            var friendship =  _friendshipRepository.GetByIdAsync(requestId);
+            if (friendship == null)
+                throw new Exception("Friend request not found");
+
+            if (friendship.Result.Status != FriendshipStatus.Pending)
+                throw new Exception("Friend request is not pending");
+
+            friendship.Result.Status = FriendshipStatus.Accepted;
+            return _friendshipRepository.UpdateAsync(friendship.Result);


The Result property is being accessed synchronously on an async operation without await. This should be: var friendship = await _friendshipRepository.GetByIdAsync(requestId, cancellationToken); and then check if null/update status without .Result.

Copilot · 2026-02-20T12:27:48Z

frontend/socialnetwork.client/src/pages/Settings.jsx

+    const [passwordConfirm, setPasswordConfirm] = useState('');
+
+    const [newEmail, setNewEmail] = useState('');
+    const [Password, setPassword] = useState('');


Inconsistent naming: variable name should be lowercase to match the naming convention used for other state variables in this file. The variable Password should be password.

Copilot · 2026-02-20T12:27:48Z

frontend/socialnetwork.client/src/pages/Register.jsx

        setError(null);
        try {
-            const res = await fetch(`${API_BASE}/api/Auth/register`, {
+            const res = await authFetch(`${API_BASE}/api/Auth/register`, {


The authFetch function is being used for registration endpoint, but registration typically doesn't require authentication. This may cause issues if the Authorization header is being set when it shouldn't be. Consider using regular fetch for the registration endpoint.

Copilot · 2026-02-20T12:27:48Z

backend/SocialNetwork.Application/DTO/Auth/LoginResponce.cs

+    public record LoginResponce
+    {
+        public string AccessToken { get; set; }
+        public string RefreshToken { get; set; }
+
+        public LoginResponce(string accessToken, string refreshToken)


Spelling error: "LoginResponce" should be "LoginResponse". This typo appears in multiple files across the codebase including DTOs, services, and interfaces.

Suggested change

public record LoginResponce

{

public string AccessToken { get; set; }

public string RefreshToken { get; set; }

public LoginResponce(string accessToken, string refreshToken)

public record LoginResponse

{

public string AccessToken { get; set; }

public string RefreshToken { get; set; }

public LoginResponse(string accessToken, string refreshToken)

Copilot · 2026-02-20T12:27:49Z

backend/SocialNetwork.API/Controllers/AuthController.cs

+        public async Task<IActionResult> LoginWithLogin([FromBody] string idToken, CancellationToken cancellationToken = default)
+        {
+            _logger.LogInformation("Google login endpoint called");
+            var responce = await _authService.LoginWithGoogleAsync(idToken, cancellationToken);


The method receives a string idToken parameter from the request body but the DTO GoogleAuthDto is not being used. Either remove the unused DTO or change the parameter to use it: [FromBody] GoogleAuthDto dto and access dto.IdToken.

Suggested change

public async Task<IActionResult> LoginWithLogin([FromBody] string idToken, CancellationToken cancellationToken = default)

{

_logger.LogInformation("Google login endpoint called");

var responce = await _authService.LoginWithGoogleAsync(idToken, cancellationToken);

public async Task<IActionResult> LoginWithLogin([FromBody] GoogleAuthDto dto, CancellationToken cancellationToken = default)

{

_logger.LogInformation("Google login endpoint called");

var responce = await _authService.LoginWithGoogleAsync(dto.IdToken, cancellationToken);

Reaction bar in frontend

Den1s-coder and others added 30 commits December 6, 2025 18:50

Merge pull request #96 from Den1s-coder/developer

c41f264

feat: add azure blob storage and profile user photo update (#67)

1420b06

feat: sort messages and split by users

cf43f65

chore: update chat page

9a84d36

feat: split chat messages on current user and others

4a4d7bb

feat: add avatar picture in profile page and change avatar button

42e1fd2

chore: update saved image url in DB

4258280

feat: add FriendService and Dto

b0b844c

feat: add FriendService and Dto (#101)

07ac483

feat: Realise Friend service

7878052

feat: add Friend controller

b6c5366

feat: Realise Friend proffile

d520acc

chore: add Created to Friendship

2bcbdb4

feat: add GetUserFriendshipsAsync method

1c6badc

feat: Realise Friend service (#102)

c7c33dc

feat: Realise Friend controller

249ae83

chore: Remove username from DTO

21680e5

chore: Remove CreatedAt from FriendShip because BaseEntity already ha…

851bca3

…ve it

Feat: Realise friend controller (#103)

2c03eda

chore: add DI for Friend repository and service

48b3f67

feat: Clicking on the username redirects to the user's profile page

b017a75

feat: Add another user page (#64)

8603e14

chore: add route for another user profile

909efd2

feat: other user profile (#64)

74adcd0

chore: add Login responce with tokens

40c9cf8

feat: update AuthService for work with Refresh Tokens

1805949

feat: add RefreshToken Entity (#75)

b4c50d2

feat: add GenearateRefreshToken method in JwtProvider (#75)

76ee1e0

feat: add RefreshToken Repository (#75)

0d52ce8

chore: RefreshToken migrations

f7ffe17

Den1s-coder and others added 21 commits February 4, 2026 12:50

feat: realise settings page (#118)

262eff4

Notifications&Settings update (#120 #119 #118)

cadf69a

chore: add Google auth nuget package

505694d

feat: add google login to login page

e2a7733

feat: add google login endpoint

14d7358

chore: google login dto return token

70e9f76

feat: add google lolgin method in auth service

2571ebd

chore: move entities to folders

3ae96f6

feat: add Reaction entities

85fb5c9

chore: update .gitignore

2bac1ba

chore: change usings

175abb0

feat: add toogle endpoints for reactions for posts,comments and messages

94f88a4

chore: add request for reaction

c8fc956

feat: add toogle reaction methods for post,comment and message services

34afaec

feat: add Toogle reaction methods for post, comment and message repos…

709c117

…itory

chore: reaction migrations

2dd9278

feat: reaction configurations

a10424b

feat: Add ReactionSummary DTO

6490797

chore: add reactions to PostDto

7b82c9e

feat: add google button for login and register

e6cd57b

Google auth and Reactions backend (#128)

b9e2be9

Copilot AI review requested due to automatic review settings February 20, 2026 12:25

Copilot started reviewing on behalf of Den1s-coder February 20, 2026 12:25 View session

Copilot AI reviewed Feb 20, 2026

View reviewed changes

Den1s-coder and others added 6 commits February 23, 2026 15:53

chore: update post mapping

92d1708

chore: seed reactions

49ea00e

feat: add include for reactions in posts

7a439d9

feat: reactions bar for interaction with reactions

baa1aab

feat: add reaction bar to posts

a043b06

Reaction bar in frontend (#130)

6fd076a

Reaction bar in frontend

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

1#98

1#98
Den1s-coder wants to merge 133 commits intodeveloperfrom
master

Den1s-coder commented Dec 6, 2025

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Copilot AI Feb 20, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Den1s-coder commented Dec 6, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants