-
Notifications
You must be signed in to change notification settings - Fork 151
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
Files in PDFs always point to pdf.sysreptor.com #259
Comments
Where do you come across this link in a rendered PDF? Pasting files (except images) is not meant to be supported in SysReptor (and should result in an error). Yet, there is an open ticket for embedding PDFs (but currently not on our roadmap) #41 (pdf.sysreptor.com is not a leftover from the cloud version, as we don't use this domain for our cloud customers. It's a "fake URL" which is set for the rendering process.) |
A colleague added some certificates into the notes of a report via copy and paste (p12 files) which end up under /files/name/xxxxxxxx.p12. |
I see, you hackers :) Yes, this is currently an unsupported scenario but is on our list which we will prioritize for H2 (see #240) |
Let's track attaching files to PDFs in a separate issue #41 |
Reopening this issue, as we should address this. If a user copies the link to a file from notes to the report, SysReptor renders this as a link to pdf.sysreptor.com in the report: This link leads to nowhere (and previously, there was an easter egg), which is an unintended behavior. A pro customer sent a report with an allegedly attached document to a customer. The customer clicked the link and landed at the page with our easter egg, which is not a pleasant situation. I think that we have the following options:
|
I noticed a weird bahviour in PDFs:
When you add a file to a finding via copy & paste it's shown as a link in the preview. When you generate a PDF the link alltough always points to
https://pdf.sysreptor.com/files/name/FILENAME
which is a leftover from the cloud version I guess.Maybe it would be possible to add those files as a PDF attachment / embed them directly into the PDF?
The text was updated successfully, but these errors were encountered: