Skip to content

Commit

Permalink
Add back setting s3 keys for non image data, fixes #1199
Browse files Browse the repository at this point in the history
  • Loading branch information
tischi committed Dec 12, 2024
1 parent 2636091 commit 1d70053
Show file tree
Hide file tree
Showing 3 changed files with 62 additions and 1 deletion.
2 changes: 1 addition & 1 deletion pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@

<groupId>org.embl.mobie</groupId>
<artifactId>mobie-viewer-fiji</artifactId>
<version>5.5.8-SNAPSHOT</version>
<version>5.5.8</version>

<!-- mvn clean install -Dmaven.test.skip=true -Dscijava.app.directory=/Users/tischer/Desktop/Fiji/Fiji-MoBIE.app -->
<!-- ../scijava-scripts/release-version.sh - -skip-version-check - -skip-license-update -->
Expand Down
8 changes: 8 additions & 0 deletions src/main/java/org/embl/mobie/MoBIE.java
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
import org.embl.mobie.io.ImageDataFormat;
import org.embl.mobie.io.imagedata.ImageData;
import org.embl.mobie.io.util.IOHelper;
import org.embl.mobie.io.util.S3Utils;
import org.embl.mobie.lib.*;
import org.embl.mobie.lib.bdv.BdvViewingMode;
import org.embl.mobie.lib.data.*;
Expand Down Expand Up @@ -116,6 +117,13 @@ public MoBIE( String uri, MoBIESettings settings ) throws IOException
this.settings = settings;
this.projectLocation = uri;

if ( MoBIEHelper.notNullOrEmpty( settings.values.getS3AccessAndSecretKey()[0] ) )
{
// This is not needed anymore for image data, but for other data types
// For image data this is now part of ImageDataFormat
S3Utils.setS3AccessAndSecretKey( settings.values.getS3AccessAndSecretKey() );
}

if ( settings.values.getProjectType().equals( ProjectType.CollectionTable ) )
{
IJ.log("\n# MoBIE" );
Expand Down
53 changes: 53 additions & 0 deletions src/test/java/debug/DebugIssue1199.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,53 @@
/*-
* #%L
* Fiji viewer for MoBIE projects
* %%
* Copyright (C) 2018 - 2024 EMBL
* %%
* Redistribution and use in source and binary forms, with or without
* modification, are permitted provided that the following conditions are met:
*
* 1. Redistributions of source code must retain the above copyright notice,
* this list of conditions and the following disclaimer.
* 2. Redistributions in binary form must reproduce the above copyright notice,
* this list of conditions and the following disclaimer in the documentation
* and/or other materials provided with the distribution.
*
* THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
* AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
* IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
* ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDERS OR CONTRIBUTORS BE
* LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
* CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
* SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
* INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
* CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
* ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
* POSSIBILITY OF SUCH DAMAGE.
* #L%
*/
package debug;

import net.imagej.ImageJ;
import org.embl.mobie.MoBIE;
import org.embl.mobie.MoBIESettings;
import org.embl.mobie.command.open.project.OpenMoBIEProjectCommand;
import org.embl.mobie.command.open.project.OpenMoBIEProjectExpertCommand;
import org.embl.mobie.command.open.project.OpenMoBIEProjectWithS3CredentialsCommand;

import java.io.IOException;

public class DebugIssue1199
{
public static void main( String[] args ) throws IOException
{
final ImageJ imageJ = new ImageJ();
imageJ.ui().showUI();

OpenMoBIEProjectWithS3CredentialsCommand command = new OpenMoBIEProjectWithS3CredentialsCommand();
command.uri = "https://s3.gwdg.de/fruitfly-larva-em";
command.s3AccessKey = "a";
command.s3SecretKey = "b";
command.run();
}
}

0 comments on commit 1d70053

Please sign in to comment.