s-stanley / ft_irc Goto Github PK

Quand un utilisateur PART d'un channel la commande fonctionne mais l'erreur NOSUCHCHANNEL lui est quand même renvoyée

Quand un 2e client se connect, et qu'il ne tape pas son password, la fonctionnalite fonctionne bien par contre le message qui lui demande son password n'est pas envoye

Quand un client se connecte, il doit envoyer son username avec la commande USER et son nickname avec la commande NICK, il faut ensuite parser et mettre dans la liste chainé.

La structure pourrait ressembler a ceci:

struct user {
  int socket_id;
  std::string nickname;
  std::string username;
  struct user *next;
};

Ressources: https://oramind.com/tutorial-how-to-make-an-irc-server-connection/

Message d'erreur: Irssi: Not connected to server
Il y a peut-être un problème avec les fds

1. Le premier user connecté ?
2. Set une variable bool is_operator dans la structure user

Remplacé:

• _atoi
• _close
• _exit
• _perror
• _read
• _strcmp
• _strlen

Permettre de :

• set un away message
• dire qu'on est away
• dire qu'on est revenu
• envoyer un message auto quand un utilisateur est away

Par exemple avec la command QUIT, lorsqu'on supprime un utilisateur de la liste qui se trouve à la fin tout se passe bien, par contre quand il a un next il y a un problème avec la liste d'utilisateurs

Je n'ai pas encore tout saisi mais il me semble qu'il faudrait stocker tous les comptes operators dans un fichier de config sous forme d'oper block: (https://www.unrealircd.org/docs/Oper_block) Qu'est ce que tu en penses ?

Il faudrait peut-être reformer le message avant de l'envoyer

Avec 2 utilisateurs connectés: si le 2eme à s'être connecté JOIN un channel et qu'il écrit un message dedans son propre message lui est envoyé 2 fois au lieu d'1 avec pour le 1er message en nick RPL WELCOME et pour le 2eme son nick. Quand le 1er utilisateur join le channel même chose et ils ne reçoivent pas les messages de l'un ou de l'autre

Le but de cette issue est de définir un plan, une liste de taches a réaliser pour réaliser ce projet.

Pour le moment je pense à:

• Setup le folder avec Makefile + main.cpp (#2)
• Créer un basic serveur qui accept les connexions (#4)
• Ajouter la vérification des arguments (port + password) et prévenir les erreurs + ajouter les tests nécessaires (#3)
• Stocker les commandes et leur fonction associee dans une map (#10)
• Stocker tous les clients dans une list chainée (#5)
• Tester que le serveur irc reçoit bien les packets envoyées par le client (#6)
• Permettre qu'un message reçu peut-être envoyer en broadcast a tous les clients (#7)
• #13
• #14
• Filter les commandes et permettre de réaliser d'autres options (envoyées un message privé, changer son username, etc...)
• Avoir un type d'utilisateur operator avec des droits différents

Il ne doit y avoir que 3 arguments: PRIVMSG destinataire message

Dans la fonction membre server::exec nous faisons beaucoup de return, cependant nous ne liberons pas la memoire du split de la commande (comme c'est le cas en bas de la fonction avec delete[] value). Ce qui genere probablement des leaks

Au debut du programme stocker toutes les commandes ainsi que leur fonction dans une map et pouvoir appeler chacune d'elles en fonction de la commande recue

• ajouter les tests nécessaires

• Quand un utilisateur a quitter un channel, il peut toujours encore des messages dedans. Ça ne devrait pas etre possible
• Quand un channel n'existe pas, un message est quand meme envoye, ça ne devrait pas etre possible non plus

1. Recuperer la commande kill
2. Regarder si il l'utilisateur est operator

Ca viendrait de la boucle qui envoie un message lorsqu'un utilisateur rejoint un channel:

for (int it = 0; it < (chan->nb_users); it++)
{
    send_user_joined_channel(
        fds[chan->users_id[it]].fd,
        get_user(i -1, all_users)->nickname,
        get_user(i -1, all_users)->username,
        value[1]
    );
}

Pour le moment on utilise exit mais il faut utiliser le protocol irc

https://fr.acervolima.com/programmation-de-sockets-en-c-c/

Le buffer utilisé pour stocker les données lues avec la fonction read() ne se vide pas.

Fix possible : Utiliser malloc et free pour l'allocation du buffer, augmenter la taille maximale

Verifier que la premiere commande entree par l'utilisateur soit PASS pour s'authentifier auprès du serveur

Au bout d'un certain temps il y a un bug sur le server, poll ne fait qu'envoyer des changements de FD et ne bloque rien. Il est possible qu'il soit necessaire de retirer les fds une fois qu'ils ont renvoyer un event.

https://i.imgur.com/KIkPERZ.png

Par contre s'il se reconnecte ou qu'un autre utilisateur se connecte avant qu'un message soit envoyé tout fonctionne normalement

Dans le cas où un nickname existe déjà il faut renvoter une erreur. Cela fonctionne bien avec telnet mais pas avec irssi qui ne recoit pas le message (ou ne l'affiche pas).

Quand un utilisateur exécute la commande QUIT ou se fait KILL il faut également le retirer de la liste des utilisateurs de chaque channel dont il fait partie.

Quand on recupere les commandes envoyes par le client (et via telnet), le split fonctionne mal, exemple

PASS      password

au lieu de

PASS password

Renvoie ERR_NEEDMOREPARAMS alors qu'il devrait valider la commande

Utiliser une liste pour stocker tous nos channels qui seraient sous forme de structure avec pour attributs leur nom et un tableau contenant tous les membres du channel.

struct channel
{
     std::string name;
     user*       users;
}

std::list<channel> channels;

Quand on envoie un message a un utilisateur, irssi affiche par exemple.

<user> :<message> is away:

Il faut modifier pour quelque chose comme

<user> is away: <message>