docu and logging

Issue #122
imixs · Jul 5, 2022 · 5af80ca · 5af80ca
1 parent af93287
commit 5af80ca
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 9 deletions.
diff --git a/imixs-adapters-wopi/README.md b/imixs-adapters-wopi/README.md
@@ -42,9 +42,9 @@ To setup the Imixs-WOPI Adapter the following environment variables must be set:
 
 | Variable              | Description  							| Example |
 | --------------------- |---------------------------------------|---------|
-| WOPI_PUBLIC_ENDPOINT  | Public client endpoint to be called by the web application. This endpoint should be SSL encrypted |https://libreoffice.foo.com/loleaflet/6a844e4/loleaflet.html?
+| WOPI_PUBLIC_ENDPOINT  | Public client endpoint to be called from the Web Browser to open the office application. This endpoint should be SSL encrypted in production |https://openoffice.foo.com
 | WOPI_HOST_ENDPOINT    | Internal Wopi Host endpoint is called by the Wopi Client to fetch and store file data. This endpoint should not be public accessible | http://my-app:8080/api/wopi/
-| WOPI_DISCOVERY_ENDPOINT | Optional public discovery endpoint used by the Wopi Host implementation to resolve the public wopi endpoint dynamically. This variable should only be set if no WOPI_PUBLIC_ENDPOINT was defined! | http://localhost:9980/hosting/discovery
+| WOPI_DISCOVERY_ENDPOINT | Internal discovery endpoint used by the Wopi Host implementation to resolve the public wopi endpoint dynamically. | http://localhost:9980/hosting/discovery
 | WOPI_FILE_EXTENSIONS   | Optional comma separated list of file extensions to be supported. |.odt,.doc,.docx,.ods,.xls,.xlsx,.ppt,.pptx|     
 | WOPI_FILE_CACHE        | file path to cache wopi files temporarily on the wop host | default: /tmp/wopi/
 | WOPI_POSTMESSAGEORIGIN | Optional postMessageOrigin  | e.g. http://application.foo.com:8080

diff --git a/imixs-adapters-wopi/src/main/java/org/imixs/workflow/wopi/WopiAccessHandler.java b/imixs-adapters-wopi/src/main/java/org/imixs/workflow/wopi/WopiAccessHandler.java
@@ -403,9 +403,7 @@ private String resolvePublicEndpoint(String uri) {
                     publicEndpoint = publicEndpoint.substring(0, publicEndpoint.length() - 1);
                 }
                 result = publicEndpoint + internalFile;
-
-                logger.info("resolved public Endpint: " + result);
-
+                logger.fine("resolved public Endpint: " + result);
             } catch (MalformedURLException e) {
                 e.printStackTrace();
             }

diff --git a/imixs-adapters-wopi/src/main/resources/javascript/imixs-wopi.js b/imixs-adapters-wopi/src/main/resources/javascript/imixs-wopi.js
@@ -163,10 +163,10 @@ IMIXS.org.imixs.workflow.wopi = (function() {
 	        // extract token from uri
 	        var tokenPos=actionuri.indexOf('access_token=');
 	        var access_token=actionuri.substring(tokenPos+13);
-	        console.log(' actionuri='+actionuri);	        
-	        console.log(' access_token='+access_token);	     
+	        // console.log(' actionuri='+actionuri);	        
+	        // console.log(' access_token='+access_token);	     
 	        actionuri=actionuri.substring(0,tokenPos);   
-			console.log(' short actionuri='+actionuri);	        
+			// console.log(' short actionuri='+actionuri);	        
 
 
 			/* Old Construction */
@@ -180,7 +180,6 @@ IMIXS.org.imixs.workflow.wopi = (function() {
 			var uiModeCollabora = '<input name="ui_defaults" value="TextSidebar=false;SpreadsheetSidebar=false" type="hidden"/>';
 			iframe.document.write('<html><body><form action="'+actionuri+'" enctype="multipart/form-data" method="post" id="libreoffice-form" style="display:none;">' +uiHiddenFields+ uiModeCollabora + '<input type="submit" value="Load..." /></form></body></html>');
 			iframe.document.close();
-
 		},
 
 		// close the office viewer and show the form part again