- Ajout système retournant automatiquement l'erreur impémentée par la commande si le SBE requis par le résultat de celle-ci n'est pas connecté au FFE

This commit is contained in:
Benjamin 2022-03-13 19:51:45 +01:00
parent 0424a2f53a
commit e5628942a2
13 changed files with 101 additions and 42 deletions

View File

@ -77,4 +77,8 @@ public class Task {
public Context getContext() { public Context getContext() {
return this.context; return this.context;
} }
public String getDomain() {
return context.getDomain();
}
} }

View File

@ -115,16 +115,23 @@ public class ClientHandler implements Runnable, AutoCloseable {
ProtocolWriter.ProtocolResult writerCommand = ruleResult.getResultCommand(); ProtocolWriter.ProtocolResult writerCommand = ruleResult.getResultCommand();
if (ruleResult.getReceiver() == ProtocolReader.ResultCmdReceiver.STOREBACKEND) { if (ruleResult.getReceiver() == ProtocolReader.ResultCmdReceiver.STOREBACKEND) {
fileFrontEnd.newCommand(context, writerCommand); // Envoie dans la file de tâche FileFrontEnd en attente d'un traitement d'un StorBackEnd // TODO : Vérifier que le StorBackEnd demandé (Pas toujours demandé) est disponible
if (ruleResult.getRequestDomain() != null && !fileFrontEnd.canExecuteCommand(ruleResult.getRequestDomain())) {
writer.print(ruleResult.onNotExecutable(context)); // Renvoie au client
writer.flush();
} else {
fileFrontEnd.newCommand(context, writerCommand); // Envoie dans la file de tâche FileFrontEnd en attente d'un traitement d'un StorBackEnd
// Attend la fin de la réalisation de la tâche // Attend la fin de la réalisation de la tâche
waitTaskResponse(); waitTaskResponse();
writer.write(response.getCommand()); // Renvoie au client
writer.flush();
response.write(this.client.getOutputStream()); // Ecrit au client si nécessaire
}
writer.write(response.getCommand()); // Renvoye au client
writer.flush();
response.write(this.client.getOutputStream()); // Ecrit au client si nécessaire
} else { } else {
writer.print(writerCommand.getCommand()); // Renvoye au client writer.print(writerCommand.getCommand()); // Renvoie au client
writer.flush(); writer.flush();
} }
@ -133,7 +140,6 @@ public class ClientHandler implements Runnable, AutoCloseable {
accessDenied(); accessDenied();
} }
} catch (IOException e) { } catch (IOException e) {
e.printStackTrace();
repository.disconnect(this); repository.disconnect(this);
break; break;
} }
@ -195,13 +201,13 @@ public class ClientHandler implements Runnable, AutoCloseable {
/** /**
* Vérifie s'il s'âgit d'une demande de déconnexion * Vérifie s'il s'âgit d'une demande de déconnexion
* @param ruleResult * @param ruleResult
*/
private void checkSignError(ProtocolWriter.ProtocolResult ruleResult) { private void checkSignError(ProtocolWriter.ProtocolResult ruleResult) {
if (ruleResult.getCommand().startsWith(SignErrorRule.NAME)) { if (ruleResult.getCommand().startsWith(SignErrorRule.NAME)) {
System.out.println("Pas pu connecter"); System.out.println("Pas pu connecter");
repository.disconnect(this); repository.disconnect(this);
} }
} }
*/
/** /**
* Permet au Client d'attendre la fin de la réalisation de sa tâche * Permet au Client d'attendre la fin de la réalisation de sa tâche

View File

@ -83,6 +83,10 @@ public class StoreProcessor extends Thread implements AutoCloseable {
this.client_run = true; this.client_run = true;
while (this.client_run) { while (this.client_run) {
try { try {
if (!store.isConnected()) {
// TODO : Gérer déconnection
break;
}
waitAction(); waitAction();
System.out.println("[SBE] Envoie commande : " + protocolResult.getCommand()); System.out.println("[SBE] Envoie commande : " + protocolResult.getCommand());
@ -104,7 +108,9 @@ public class StoreProcessor extends Thread implements AutoCloseable {
alertAvalaible(responseResult.getResultCommand()); alertAvalaible(responseResult.getResultCommand());
} catch (IOException ignore) { } } catch (IOException ignore) {
// TODO : Gérer déconnection
}
} }
} }
@ -153,6 +159,7 @@ public class StoreProcessor extends Thread implements AutoCloseable {
try { try {
this.client_run = false; this.client_run = false;
this.store.close(); this.store.close();
// TODO : Gérer déconnection (enlever du repo et prévenir client et FileFrontEnd)
} catch (IOException ignored) { } } catch (IOException ignored) { }
} }
} }

View File

@ -18,4 +18,5 @@ public interface ClientHandlerFFE {
*/ */
void newCommand(Context context, ProtocolWriter.ProtocolResult command); void newCommand(Context context, ProtocolWriter.ProtocolResult command);
boolean canExecuteCommand(String domain);
} }

View File

@ -24,4 +24,6 @@ public interface MulticastSPR {
String findDomain(Task task); String findDomain(Task task);
void assignTask(String stor, Task task); void assignTask(String stor, Task task);
boolean hasDomain(String domain);
} }

View File

@ -26,6 +26,7 @@ public abstract class ProtocolReader {
STOREBACKEND STOREBACKEND
} }
/** /**
* Modèle utilisé par tout les résultats des règles de protocol. * Modèle utilisé par tout les résultats des règles de protocol.
* Lorsqu'il retourne son résultat, on vérifie si il y a une demande de lecture/écriture de fichier depuis le réseau. Si oui on appel ces méthodes, sinon on ne fait rien. * Lorsqu'il retourne son résultat, on vérifie si il y a une demande de lecture/écriture de fichier depuis le réseau. Si oui on appel ces méthodes, sinon on ne fait rien.
@ -42,6 +43,11 @@ public abstract class ProtocolReader {
*/ */
private final Context context; private final Context context;
/**
* Domaine demandé pour l'exécution de cette commande (NULL si aucun domaine spécifique demandé)
*/
private String requestDomain;
public ProtocolResult(Context context) { public ProtocolResult(Context context) {
this.context = context; this.context = context;
} }
@ -58,7 +64,7 @@ public abstract class ProtocolReader {
* *
* @return Receiver * @return Receiver
*/ */
public ResultCmdReceiver getReceiver() { public final ResultCmdReceiver getReceiver() {
return receiver; return receiver;
} }
@ -67,7 +73,7 @@ public abstract class ProtocolReader {
* *
* @return Commande * @return Commande
*/ */
public ProtocolWriter.ProtocolResult getResultCommand() { public final ProtocolWriter.ProtocolResult getResultCommand() {
return this.resultCommand; return this.resultCommand;
} }
@ -77,11 +83,12 @@ public abstract class ProtocolReader {
* @param resultCommand Commande à envoyer * @param resultCommand Commande à envoyer
* @param receiver Le receveur de cette commande * @param receiver Le receveur de cette commande
*/ */
public void setResultCommand(ProtocolWriter.ProtocolResult resultCommand, ResultCmdReceiver receiver) { public final void setResultCommand(ProtocolWriter.ProtocolResult resultCommand, ResultCmdReceiver receiver) {
this.resultCommand = resultCommand; this.resultCommand = resultCommand;
this.receiver = receiver; this.receiver = receiver;
} }
/** /**
* Permet de lire un fichier. Cad reçevoir le contenu d'un fichier provenant du réseau. * Permet de lire un fichier. Cad reçevoir le contenu d'un fichier provenant du réseau.
* Redéfinissez cette méthode pour l'utiliser * Redéfinissez cette méthode pour l'utiliser
@ -96,10 +103,32 @@ public abstract class ProtocolReader {
* *
* @return Context courant * @return Context courant
*/ */
protected Context getContext() { protected final Context getContext() {
return context; return context;
} }
/**
* Cette méthode est appelée lorsque la commande à exécuter ne peut pas être exécutée
*/
public final String onNotExecutable(Context context) {
ProtocolResult cmdResult = onError(context);
return cmdResult == null ? null : cmdResult.getResultCommand().getCommand();
}
/**
* Accesseur du domaine demandé pour l'exécution de cette commande (NULL si aucun domaine spécifique demandé)
*/
public String getRequestDomain() {
return requestDomain;
}
/**
* Mutateur du domaine demandé pour l'exécution de cette commande (NULL si aucun domaine spécifique demandé)
*/
public void setRequestDomain(String requestDomain) {
this.requestDomain = requestDomain;
}
} }
/** /**

View File

@ -60,6 +60,7 @@ public abstract class ProtocolWriter {
public Context getContext() { public Context getContext() {
return context; return context;
} }
} }
/** /**

View File

@ -4,9 +4,12 @@ import lightcontainer.domains.client.Context;
import lightcontainer.interfaces.ProtocolRepository; import lightcontainer.interfaces.ProtocolRepository;
import lightcontainer.protocol.ProtocolReader; import lightcontainer.protocol.ProtocolReader;
import lightcontainer.protocol.rules.writer.GetFileErrorRule; import lightcontainer.protocol.rules.writer.GetFileErrorRule;
import lightcontainer.protocol.rules.writer.GetFileOkRule;
import lightcontainer.storage.File; import lightcontainer.storage.File;
import lightcontainer.storage.ReadOnlyFile; import lightcontainer.storage.ReadOnlyFile;
import java.util.Iterator;
public class GetFileRule extends ProtocolReader { public class GetFileRule extends ProtocolReader {
// Constants // Constants
private static final String PATTERN = "^GETFILE ([^ !]{1,20})\r\n$"; private static final String PATTERN = "^GETFILE ([^ !]{1,20})\r\n$";
@ -50,6 +53,10 @@ public class GetFileRule extends ProtocolReader {
ReadOnlyFile file = context.getFileOf(data[FILE_NAME], context.getLogin()); ReadOnlyFile file = context.getFileOf(data[FILE_NAME], context.getLogin());
// Précision du store back end demandé pour traiter cette commande.
String requestDomain = extractRequestDomain(file.getStorageIterator());
result.setRequestDomain(requestDomain);
if (true) { if (true) {
} else { } else {
@ -68,4 +75,14 @@ public class GetFileRule extends ProtocolReader {
result.setResultCommand(protocolRep.executeWriter(context, GetFileErrorRule.NAME), ResultCmdReceiver.CLIENT); result.setResultCommand(protocolRep.executeWriter(context, GetFileErrorRule.NAME), ResultCmdReceiver.CLIENT);
return result; return result;
} }
/**
* TODO : But futur est de pouvoir en avoir plusieurs
* Cette méthode permet de choisir le domaine voulu.
* @param storageIterator Les domaines
* @return Le domain choisi
*/
private String extractRequestDomain(Iterator<String> storageIterator) {
return storageIterator.next();
}
} }

View File

@ -102,6 +102,8 @@ public class SavefileRule extends ProtocolReader {
@Override @Override
protected SavefileRule.Result onExecuted(Context context, String... data) { protected SavefileRule.Result onExecuted(Context context, String... data) {
SavefileRule.Result result = new SavefileRule.Result(context, data[FILE_NAME], Integer.parseInt(data[FILE_SIZE])); SavefileRule.Result result = new SavefileRule.Result(context, data[FILE_NAME], Integer.parseInt(data[FILE_SIZE]));
return result; return result;
} }

View File

@ -11,7 +11,7 @@ import java.io.OutputStream;
*/ */
public class SendfileRule extends ProtocolWriter { public class SendfileRule extends ProtocolWriter {
private static final String PATTERN = "^SENDFILE [A-Za-z0-9.]{0,200} [0-9]{1,10} [A-Za-z0-9.]{50,200}\r\n$"; private static final String PATTERN = "^SENDFILE [A-Za-z0-9.]{50,200} [0-9]{1,10} [A-Za-z0-9.]{50,200}\r\n$";
public static final String NAME = "SENDFILE"; public static final String NAME = "SENDFILE";

View File

@ -82,4 +82,9 @@ public class FileFrontEnd implements ClientHandlerFFE, StoreProcessorFFE {
tasks.add(task); tasks.add(task);
alertStoreProcessors(task); alertStoreProcessors(task);
} }
@Override
public boolean canExecuteCommand(String domain) {
return storeRepository.hasDomain(domain);
}
} }

View File

@ -78,6 +78,17 @@ public class StoreProcessorRepository implements AutoCloseable, MulticastSPR {
handler.executeCommand(task.getContext(), task.getCommand()); handler.executeCommand(task.getContext(), task.getCommand());
} }
@Override
public boolean hasDomain(String domain) {
for (StoreProcessor handler : handlers) {
if (handler.getDomain().equals(domain)) {
return true;
}
}
return false;
}
/** /**
* AutoClosable Function * AutoClosable Function
* Closes all StoreProcessor stored in this repository and deallocates all resources. * Closes all StoreProcessor stored in this repository and deallocates all resources.

View File

@ -1,27 +1 @@
{ {"unicast_port":8000,"multicast_ip":"224.66.66.1","multicast_port":15502,"network_interface":"My network interface","tls":true,"storagePath":"/home/benjamin/ffe","users":[{"name":"aaaaa","password":"5d628c274ebb008324f1e199d3bfff0a3fe839730a7f2355e82850d7acca5e5ca64db9071abf3d91034295695f84a617","aes_key":"qlTH6TijnfMRnrS0Qf+k6IPKGp5LoRMXGxCq16e+mF4=","passwordSalt":"Ns8Al6DpqPsIDlCSRBVTEg==","files":[{"name":"main.py","fileNameSalt":"IJNYL681pFqbF9OHzRuHIg==","size":854,"iv":"bPCnwYbenKvFfwbhq+HI5A==","storage":["lightcontainerSB01"]},{"name":"README.md","fileNameSalt":"/jo0zYyQs96gWI9OgBXiPQ==","size":17,"iv":"rvOFhgEvgFMISO44jqlSRg==","storage":["lightcontainerSB01"]}]}]}
"unicast_port": 8000,
"multicast_ip": "224.66.66.1",
"multicast_port": 15502,
"network_interface": "My network interface",
"tls": true,
"storagePath": "/home/benjamin/ffe",
"users": [
{
"name": "aaaaa",
"password": "5d628c274ebb008324f1e199d3bfff0a3fe839730a7f2355e82850d7acca5e5ca64db9071abf3d91034295695f84a617",
"aes_key": "qlTH6TijnfMRnrS0Qf+k6IPKGp5LoRMXGxCq16e+mF4=",
"passwordSalt": "Ns8Al6DpqPsIDlCSRBVTEg==",
"files": [
{
"name": "README.md",
"fileNameSalt": "/jo0zYyQs96gWI9OgBXiPQ==",
"size": 17,
"iv": "rvOFhgEvgFMISO44jqlSRg==",
"storage": [
"lightcontainerSB01"
]
}
]
}
]
}