-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
password imagej log #136
Comments
This issue has been mentioned on Image.sc Forum. There might be relevant details there: |
I'm not aware of any change with that respect either. I'll try to find out what's going on there. |
Can replicate the issue. Just before the Bioformats import dialog opens, a log window pops up displaying the login credentials. I turned the debug mode on to get some more information (creds replaced by
Maybe it comes from the OmeroReader? |
This issue has been mentioned on Image.sc Forum. There might be relevant details there: https://forum.image.sc/t/omero-imagej-plugin-shows-password-in-log-window/39114/2 |
@jburel @dominikl : https://github.com/ome/omero-insight/blob/master/src/main/java/org/openmicroscopy/shoola/env/ui/TaskBarManager.java#L358 looks to be the culprit:
|
I think it is elsewhere that it is written in the log. This is the way to pass info the plugin. I am looking into the problem |
https://github.com/ome/omero-blitz/blob/master/src/main/java/loci/ome/io/OmeroReader.java#L223 is the source of the problem when using imageJ in debug mode. |
Just a question: |
insight communicates with the server using a secure and encrypted connection. I have pushed a fix for review that will prevent the string from being logged locally. We will have a new version out as soon as possible |
This issue has been mentioned on Image.sc Forum. There might be relevant details there: https://forum.image.sc/t/imagej-omero-plugins-shows-password-in-log-window/39864/1 |
https://forum.image.sc/t/opening-image-from-omero-in-imagej-creates-log-file-with-plain-text-password/38621
The text was updated successfully, but these errors were encountered: