From c27d50c575ff5867aef8f10ccf2b5f2ac43dae08 Mon Sep 17 00:00:00 2001 From: Nino Breuer Date: Fri, 28 Nov 2014 16:07:58 +0100 Subject: • changed implementation of user authentication. • changed position of class file • fixed some little things (not necessary to mention) Important Notes: •organizationID in token returns "wrong" values, therefore university is still set by client. To be fixed someday. • userID is the value with which the user starts his log in. this value is not represented in the token! (as it is not(!) the email). To be implemented someday, as this value is represented in the masterserver. • Feature isn't well tested. --- .../java/gui/image/CreateImageAllgemein_GUI.java | 10 ++++ .../java/gui/image/CreateImageTechnisch_GUI.java | 11 +++- .../src/main/java/gui/image/DeleteImage_GUI.java | 35 ++++++------ .../java/gui/image/EditImageAllgemein_GUI.java | 11 +++- .../java/gui/image/EditImageTechnisch_GUI.java | 14 +++-- .../main/java/gui/image/FTPCreateUploader_GUI.java | 61 +++++++++++---------- .../main/java/gui/image/FTPEditDownloader_GUI.java | 33 +++++++----- .../main/java/gui/image/FTPEditUploader_GUI.java | 62 ++++++++++++++-------- .../java/gui/image/FTPSearchDownloader_GUI.java | 34 ++++++++---- .../java/gui/image/PermissionCreateImage_GUI.java | 16 +++--- .../java/gui/image/PermissionEditImage_GUI.java | 19 ++++--- .../main/java/gui/image/SearchEditImage_GUI.java | 24 ++++----- .../src/main/java/gui/image/SearchImage_GUI.java | 27 +++++----- 13 files changed, 214 insertions(+), 143 deletions(-) (limited to 'dozentenmodul/src/main/java/gui/image') diff --git a/dozentenmodul/src/main/java/gui/image/CreateImageAllgemein_GUI.java b/dozentenmodul/src/main/java/gui/image/CreateImageAllgemein_GUI.java index 5d72fc36..b10000ef 100644 --- a/dozentenmodul/src/main/java/gui/image/CreateImageAllgemein_GUI.java +++ b/dozentenmodul/src/main/java/gui/image/CreateImageAllgemein_GUI.java @@ -36,13 +36,16 @@ import javax.swing.border.TitledBorder; import models.Image; import models.Links; +import models.SessionData; import models.person; +import server.generated.Server.Client; import util.GuiOrganizer; import util.OpenLinks; import javax.swing.JScrollPane; import org.apache.log4j.Logger; +import org.apache.thrift.TException; @SuppressWarnings("serial") public class CreateImageAllgemein_GUI extends JFrame { @@ -70,6 +73,7 @@ public class CreateImageAllgemein_GUI extends JFrame { "Um das Image richtig zuordnen zu können, werden hier einige grundlegende Informationen benötigt.
" + "Geben Sie daher bitte einen allgemeinen Namen, sowie eine genauere Beschreibung ihres Images (Labors) an.
" + ""; + Client client = models.Client.clientcon.getClient(); /** * Create the dialog. @@ -78,6 +82,12 @@ public class CreateImageAllgemein_GUI extends JFrame { addWindowListener(new WindowAdapter() { @Override public void windowClosing(WindowEvent arg0) { + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } // Beendet das Fenster auf X System.exit(0); } diff --git a/dozentenmodul/src/main/java/gui/image/CreateImageTechnisch_GUI.java b/dozentenmodul/src/main/java/gui/image/CreateImageTechnisch_GUI.java index b77f93ea..12ef7554 100644 --- a/dozentenmodul/src/main/java/gui/image/CreateImageTechnisch_GUI.java +++ b/dozentenmodul/src/main/java/gui/image/CreateImageTechnisch_GUI.java @@ -43,6 +43,7 @@ import javax.swing.border.TitledBorder; import models.Image; import models.Links; +import models.SessionData; import server.generated.Server.Client; import thrift.ThriftConnection; import util.GuiOrganizer; @@ -60,7 +61,7 @@ public class CreateImageTechnisch_GUI extends JFrame { * Logger instance for this class. */ private final static Logger LOGGER = Logger.getLogger(CreateImageTechnisch_GUI.class); - + private final JPanel contentPanel = new JPanel(); String[] result; JSpinner spinnerCPU; @@ -89,6 +90,12 @@ public class CreateImageTechnisch_GUI extends JFrame { addWindowListener(new WindowAdapter() { @Override public void windowClosing(WindowEvent arg0) { + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e1) { + // TODO Auto-generated catch block + e1.printStackTrace(); + } System.exit(0); } }); @@ -184,7 +191,7 @@ public class CreateImageTechnisch_GUI extends JFrame { comboBox = new JComboBox(); List os = null; try { - os = client.getAllOS(); + os = client.getAllOS(SessionData.session.getAuthToken()); for(int i=0; i list = client.getAllOS(); + List list = client.getAllOS(SessionData.session.getAuthToken()); // Leeres Item hinzufuegen comboBox.addItem(""); for (int i = 0; i < list.size(); i++) { @@ -376,7 +375,7 @@ public class DeleteImage_GUI extends JFrame { // oeffnet das Hauuetmenue con.closeThriftConnection(); - + // oeffnet das Hauuetmenue DeleteImage_GUI.this.setVisible(false); @@ -412,8 +411,7 @@ public class DeleteImage_GUI extends JFrame { String imageid = ""; String imageversion = ""; int eingabe; - String username = person.verantwortlicher.getName() + " " - + person.verantwortlicher.getVorname(); + if (tablemyImages.getSelectedRowCount() != 0) { // frage, soll das selektierte Daten geloescht werden? @@ -441,11 +439,12 @@ public class DeleteImage_GUI extends JFrame { try { if (!client.connectedToLecture(imageid, - imageversion)) { + imageversion,SessionData.session.getAuthToken())) { try { - client.deleteImageServer(imageid, - imageversion); - if (client.deleteImageData(imageid,imageversion)) { + + client.deleteImageServer(imageid,imageversion, SessionData.session.getAuthToken()); + + if (client.deleteImageData(imageid,imageversion,SessionData.session.getAuthToken())) { LOGGER.info("Image '"+imageid+"' erfolgreich gelöscht."); @@ -787,7 +786,7 @@ public class DeleteImage_GUI extends JFrame { images = client.getImageList(person.verantwortlicher.getUserID()); } else { //get permitted list - images = client.getImageListPermissionAdmin(person.verantwortlicher.getUserID()); + images = client.getImageListPermissionAdmin(person.verantwortlicher.getUserID(),SessionData.session.getAuthToken()); } Iterator i = images.iterator(); @@ -854,7 +853,7 @@ public class DeleteImage_GUI extends JFrame { try { SimpleDateFormat in = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); SimpleDateFormat out = new SimpleDateFormat("dd.MM.yyyy HH:mm:ss"); - Map res = client.getImageData(id, version); + Map res = client.getImageData(id, version,SessionData.session.getAuthToken()); labelName.setText(res.get("name")); labelOS.setText(res.get("os")); labelUpdate.setText(out.format(in.parse(res.get("lastupdate")))); diff --git a/dozentenmodul/src/main/java/gui/image/EditImageAllgemein_GUI.java b/dozentenmodul/src/main/java/gui/image/EditImageAllgemein_GUI.java index a848c805..273e3748 100644 --- a/dozentenmodul/src/main/java/gui/image/EditImageAllgemein_GUI.java +++ b/dozentenmodul/src/main/java/gui/image/EditImageAllgemein_GUI.java @@ -39,6 +39,7 @@ import javax.swing.border.TitledBorder; import models.Image; import models.ItemOwner; import models.Links; +import models.SessionData; import models.person; import org.apache.log4j.Logger; @@ -95,6 +96,12 @@ public class EditImageAllgemein_GUI extends JFrame { public void windowClosing(WindowEvent arg0) { // Beendet die Anwendung bei einem Klick auf das X con.closeThriftConnection(); + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } System.exit(0); } }); @@ -439,8 +446,8 @@ public class EditImageAllgemein_GUI extends JFrame { public void initData() { try { - Map map = client.getImageData( - Image.image.getImageId(), Image.image.getVersion()); + Map map = client.getImageData(Image.image.getImageId(), Image.image.getVersion(),SessionData.session.getAuthToken()); + if (map.get("internet").equals("0")) { Image.image.setInternet(false); } else { diff --git a/dozentenmodul/src/main/java/gui/image/EditImageTechnisch_GUI.java b/dozentenmodul/src/main/java/gui/image/EditImageTechnisch_GUI.java index f9701a0c..418ce276 100644 --- a/dozentenmodul/src/main/java/gui/image/EditImageTechnisch_GUI.java +++ b/dozentenmodul/src/main/java/gui/image/EditImageTechnisch_GUI.java @@ -41,6 +41,7 @@ import thrift.ThriftConnection; import util.OpenLinks; import models.Image; import models.Links; +import models.SessionData; import models.person; import javax.swing.JComboBox; @@ -82,7 +83,12 @@ public class EditImageTechnisch_GUI extends JFrame { @Override public void windowClosing(WindowEvent arg0) { // Beendet die Anwendung bei Klick auf das X - + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } System.exit(0); } @@ -174,7 +180,7 @@ public class EditImageTechnisch_GUI extends JFrame { comboBox = new JComboBox(); List os = null; try { - os=client.getAllOS(); + os=client.getAllOS(SessionData.session.getAuthToken()); for(int i=0;i"; public PermissionEditImage_GUI(Component formerGUI) { - - try { - client.setTokenForSession(SessionData.session.getAuthToken()); - } catch (TException e1) { - // TODO Auto-generated catch block - e1.printStackTrace(); - } - + addWindowListener(new WindowAdapter() { @Override public void windowClosing(WindowEvent arg0) { + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } System.exit(0); } }); @@ -203,7 +202,7 @@ public class PermissionEditImage_GUI extends JFrame { //System.out.println("Loading data from db"); try { - map = client.getPermissionForUserAndImage(person.verantwortlicher.getUserID(), Image.image.getImageId()); + map = client.getPermissionForUserAndImage(SessionData.session.getAuthToken(),Image.image.getImageId(),person.verantwortlicher.getUserID()); Iterator i = map.iterator(); int x = 0; diff --git a/dozentenmodul/src/main/java/gui/image/SearchEditImage_GUI.java b/dozentenmodul/src/main/java/gui/image/SearchEditImage_GUI.java index 7b8578c6..1b5feb6e 100644 --- a/dozentenmodul/src/main/java/gui/image/SearchEditImage_GUI.java +++ b/dozentenmodul/src/main/java/gui/image/SearchEditImage_GUI.java @@ -147,13 +147,6 @@ public class SearchEditImage_GUI extends JFrame { */ public SearchEditImage_GUI(Component formerGUI) { - try { - client.setTokenForSession(SessionData.session.getAuthToken()); - } catch (TException e1) { - // TODO Auto-generated catch block - e1.printStackTrace(); - } - filters.add(RowFilter.regexFilter(".", 0)); rf = RowFilter.orFilter(filters); @@ -163,7 +156,14 @@ public class SearchEditImage_GUI extends JFrame { @Override public void windowClosing(WindowEvent arg0) { // Beendet die Anwendung nach klick auf X + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } con.closeThriftConnection(); + System.exit(0); } @@ -321,7 +321,7 @@ public class SearchEditImage_GUI extends JFrame { try { // Holt sich eine Liste aller verfuegbaren Betriebssysteme vom // Server - List list = client.getAllOS(); + List list = client.getAllOS(SessionData.session.getAuthToken()); // Leeres Item hinzufuegen comboBox.addItem(""); for (int i = 0; i < list.size(); i++) { @@ -763,7 +763,7 @@ public class SearchEditImage_GUI extends JFrame { images = client.getImageList(person.verantwortlicher.getUserID()); } else { //only get permitted list - images = client.getImageListPermissionWrite(person.verantwortlicher.getUserID()); + images = client.getImageListPermissionWrite(person.verantwortlicher.getUserID(),SessionData.session.getAuthToken()); } Iterator i = images.iterator(); @@ -830,7 +830,7 @@ public class SearchEditImage_GUI extends JFrame { try { SimpleDateFormat in = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); SimpleDateFormat out = new SimpleDateFormat("dd.MM.yyyy HH:mm:ss"); - Map res = client.getImageData(id, version); + Map res = client.getImageData(id, version,SessionData.session.getAuthToken()); labelName.setText(res.get("name")); labelOS.setText(res.get("os")); labelUpdate.setText(out.format(in.parse(res.get("lastupdate")))); @@ -859,7 +859,7 @@ public class SearchEditImage_GUI extends JFrame { textAreadesc.setText(res.get("desc")); res.clear(); - List contacts = client.getAdditionalImageContacts(id); + List contacts = client.getAdditionalImageContacts(id,SessionData.session.getAuthToken()); labelWeitereVerantwortliche.setText(""); for (int i = 0; i < contacts.size(); i++) { @@ -879,7 +879,7 @@ public class SearchEditImage_GUI extends JFrame { try { //get values from server and set them in model - map=client.getItemOwner(imageid); + map=client.getItemOwner(imageid,SessionData.session.getAuthToken()); ItemOwner.itemOwner.setUserID(map.get("userID")); ItemOwner.itemOwner.setUsername(map.get("loginName")); diff --git a/dozentenmodul/src/main/java/gui/image/SearchImage_GUI.java b/dozentenmodul/src/main/java/gui/image/SearchImage_GUI.java index a0d36c5a..e9decd1b 100644 --- a/dozentenmodul/src/main/java/gui/image/SearchImage_GUI.java +++ b/dozentenmodul/src/main/java/gui/image/SearchImage_GUI.java @@ -167,20 +167,17 @@ public class SearchImage_GUI extends JFrame { */ public SearchImage_GUI(Component formerGUI) { - - - try { - client.setTokenForSession(SessionData.session.getAuthToken()); - } catch (TException e1) { - // TODO Auto-generated catch block - e1.printStackTrace(); - } - addWindowListener(new WindowAdapter() { @Override public void windowClosing(WindowEvent arg0) { // Beendet die Anwendung nach klick auf X + try { + client.setSessionInvalid(SessionData.session.getAuthToken()); + } catch (TException e) { + // TODO Auto-generated catch block + e.printStackTrace(); + } con.closeThriftConnection(); System.exit(0); } @@ -430,7 +427,7 @@ public class SearchImage_GUI extends JFrame { try { // Holt sich eine Liste aller verfuegbaren Betriebssysteme vom // Server - List list = client.getAllOS(); + List list = client.getAllOS(SessionData.session.getAuthToken()); // Leeres Item hinzufuegen comboBox.addItem(""); for (int i = 0; i < list.size(); i++) { @@ -1024,7 +1021,7 @@ public class SearchImage_GUI extends JFrame { //get complete list images = client.getImageList(person.verantwortlicher.getUserID()); } else { - images = client.getImageListPermissionRead(person.verantwortlicher.getUserID()); + images = client.getImageListPermissionRead(person.verantwortlicher.getUserID(),SessionData.session.getAuthToken()); } Iterator i = images.iterator(); @@ -1069,7 +1066,7 @@ public class SearchImage_GUI extends JFrame { List images; try { // Hole eine Liste der Images - images = client.getImageList(person.verantwortlicher.getUserID()); + images = client.getImageList(SessionData.session.getAuthToken()); Iterator i = images.iterator(); SimpleDateFormat in = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); @@ -1117,7 +1114,7 @@ public class SearchImage_GUI extends JFrame { List images; try { // Hole eine Liste der Images - images = client.getImageListAllTemplates(); + images = client.getImageListAllTemplates(SessionData.session.getAuthToken()); Iterator i = images.iterator(); SimpleDateFormat in = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); @@ -1183,7 +1180,7 @@ public class SearchImage_GUI extends JFrame { try { SimpleDateFormat in = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss"); SimpleDateFormat out = new SimpleDateFormat("dd.MM.yyyy HH:mm:ss"); - Map res = client.getImageData(id, version); + Map res = client.getImageData(id, version,SessionData.session.getAuthToken()); labelName.setText(res.get("name")); labelOS.setText(res.get("os")); labelUpdate.setText(out.format(in.parse(res.get("lastupdate")))); @@ -1210,7 +1207,7 @@ public class SearchImage_GUI extends JFrame { labelVersion.setText(res.get("version")); textAreadesc.setText(res.get("desc")); - List contacts = client.getAdditionalImageContacts(id); + List contacts = client.getAdditionalImageContacts(id,SessionData.session.getAuthToken()); labelWeitereVerantwortliche.setText(""); for (int i = 0; i < contacts.size(); i++) { -- cgit v1.2.3-55-g7522