{"action":"created","issue":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/3289","labels_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/3289/labels{/name}","comments_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/3289/comments","events_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/3289/events","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/3289","id":52221928,"number":3289,"title":"CLI error report","user":{"login":"jburel","id":1022396,"avatar_url":"https://avatars.githubusercontent.com/u/1022396?v=3","gravatar_id":"","url":"https://api.github.com/users/jburel","html_url":"https://github.com/jburel","followers_url":"https://api.github.com/users/jburel/followers","following_url":"https://api.github.com/users/jburel/following{/other_user}","gists_url":"https://api.github.com/users/jburel/gists{/gist_id}","starred_url":"https://api.github.com/users/jburel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jburel/subscriptions","organizations_url":"https://api.github.com/users/jburel/orgs","repos_url":"https://api.github.com/users/jburel/repos","events_url":"https://api.github.com/users/jburel/events{/privacy}","received_events_url":"https://api.github.com/users/jburel/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/labels/develop","name":"develop","color":"02e10c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":17,"created_at":"2014-12-17T10:10:09Z","updated_at":"2015-01-07T14:22:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/pulls/3289","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/3289","diff_url":"https://github.com/openmicroscopy/openmicroscopy/pull/3289.diff","patch_url":"https://github.com/openmicroscopy/openmicroscopy/pull/3289.patch"},"body":"In this PR:\r\n * Remove usage of commons-http\r\n * Modify submission of files: support for both http and https.\r\n * Fix submission of log file (server import log) when using --logs\r\n * If the file that failed to import the log file (if available) is also submitted.\r\n * A zip is created if number of files to submit > 1.\r\n\r\nWe use https://qa.staging.openmicroscopy.org to test this PR\r\nTest 1\r\n * Try to import  a faulty image e.g. squig/data_repo/test_images_bad/multi-image-pixels.ome.tif . \r\n * Test logs report i.e. --logs . Check that the corresponding entry has a .log file (not .zip)\r\n```\r\nbin/omero import -s localhost -u root -w omero -p 4064  --report --logs PATH_TO/multi-image-pixels.ome.tif\r\n```\r\n * Test upload report --upload Check that the corresponding entry has .zip\r\n```\r\nbin/omero import -s localhost -u root -w omero -p 4064  --report --upload PATH_TO/multi-image-pixels.ome.tif\r\n```\r\n\r\nTest 2\r\n * Try to import squig/data_repo/test_images_broken/broken_images_scenario/fails_at_processing/wrong-light-source-mk3.ome\r\n * Test logs. Make sure that there is no NPE in console and that the error is submitted. No log file\r\n * Test upload. In that case, only the file should be upload since the log is not available.\r\n\r\n\r\nThe tests under blitz will have to reviewed, To be done in another PR."},"comment":{"url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/comments/69027516","html_url":"https://github.com/openmicroscopy/openmicroscopy/pull/3289#issuecomment-69027516","issue_url":"https://api.github.com/repos/openmicroscopy/openmicroscopy/issues/3289","id":69027516,"user":{"login":"jburel","id":1022396,"avatar_url":"https://avatars.githubusercontent.com/u/1022396?v=3","gravatar_id":"","url":"https://api.github.com/users/jburel","html_url":"https://github.com/jburel","followers_url":"https://api.github.com/users/jburel/followers","following_url":"https://api.github.com/users/jburel/following{/other_user}","gists_url":"https://api.github.com/users/jburel/gists{/gist_id}","starred_url":"https://api.github.com/users/jburel/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jburel/subscriptions","organizations_url":"https://api.github.com/users/jburel/orgs","repos_url":"https://api.github.com/users/jburel/repos","events_url":"https://api.github.com/users/jburel/events{/privacy}","received_events_url":"https://api.github.com/users/jburel/received_events","type":"User","site_admin":false},"created_at":"2015-01-07T14:22:15Z","updated_at":"2015-01-07T14:22:15Z","body":"@sbesson, @pwalczysko: I found the source of the problem\r\nThe .fake file is an empty file that why it does not get uploaded.\r\n\r\n@pwalczysko: if you want to test with the existing build, just open the fake file in a text editor and a character e.g. 1. It will work.\r\nI will push a commit improving error report so it is not misleading."}}