-
Notifications
You must be signed in to change notification settings - Fork 61
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
Privacy tickbox in admin screen #42
Comments
Doesn't seem to be a big thing. I can work on this as I already did some changes in the configuration backend (see #41). |
I added a privacy option in my own server and the parameter "stealth=1" is added now in the URL, when this option is set. However, external connections are still there. I tested this with https://www.draw.io as well as with my own hosted server - no difference. In fact,
What "stealth=1" really does, is to avoid connections to Google Drive and OneDrive, see yourself: https://www.draw.io/ However - for the embedded version inside Nextcloud, Google Drive and OneDrive don't get used anyway, so |
Oh - and BTW: external services are still useable as well. PDF export is also possible with |
If you want to check my changes, if there is anything missing: https://github.com/arnowelzel/nextcloud-drawio/commit/bfbc0143a135a3c47e39ce91520f7e684d3a1d3f |
After checking https://desk.draw.io/support/solutions/articles/16000042546-what-url-parameters-are-supported- and experimenting a bit with other parameters as well, I believe the right parameter is
And indeed - when using this parameter, all external resource requests are gone. So a combination of |
@pawelrojek Obviously, you're a busy person, so given @arnowelzel's contributions is there any scope to make him a project admin (assuming Arno doesn't object) to help you with the load? |
@davidjgraph @pawelrojek No objections here. I'd be glad to help if needed. |
@arnowelzel I just checked PDF export in stealth mode, it sends me to the built-in PDF generation, this isn't using the remote PDF service. That's using latest Firefox (60.0.2) on MacOS 10.13.5 navigating to https://www.draw.io/?splash=0&stealth=1, drawing a simple diagram and selecting File->Export As->PDF. What should happen is we use the print to PDF functionality from print preview available in Chrome or MacOS. |
Stealth=1 should (TM) be the right parameter, offline=1 will store the app in the appcache and load from there first. math=0 will stop the MathJax code loading. Stealth=1 should really disable remote footer loading, that's a bug. |
@davidjgraph What "built-in PDF generation"? I just get a print dialog with no PDF at all. Just the dialog title is "PDF" and not "Print" - but it behaves exaclty like the print dialog (tested with Firefox 60.0): When I click "Print" the print dialog of my operating system appears and "Preview" will just show a preview in the browser - but this is not a PDF but HTML with embedded SVG. PS: We should continue the discussion about PDF and stealth mode to jgraph/drawio#275. |
The OS or browser PDF generation, if available. We could disable the menu option if you're not using Chrome or MacOS. If you've installed a print to PDF function, it could probably be reasonably assumed you know to use it. |
@pawelrojek I have to share Arno's original concerns a little now. You're obviously an extremely busy person, wouldn't sharing some of your workload out help that? You're the only admin to this project and to the draw.io NextCloud app entry. Under what conditions would you consider adding additional admins? |
@davidjgraph You're right and I'm really sorry for all the delays. More admins seems like the best way to go for the good of this project. |
No need to apologize, let others help you out so you don't get nagged so much 8-) |
We have users complaining to us (draw.io) that the default war we provide makes calls to external sites when they run it with NextCloud. It would be useful if the admin screen had a tickbox called something like "private mode" that added the appropriate URL parameters to the draw.io URL to switch off external connections. I think that's stealth=1, but need to double check.
We're happy to sponsor this work (pay for it).
The text was updated successfully, but these errors were encountered: