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
quotation sign defaces after edit of an entry. #304
Comments
@joerg-nafcom ah it appears that we are escaping quotes to protect against some sort of SQL injection. I think we will need to find a creative way to ensure that this is properly fixed. Thanks for the report! |
No, we don't use any SQL at all, although I thought about switching to an SQLite database in the future |
oops @emilengler i mean html injection oops. I think this probably has to do with a call to |
Yes I think this causes the problem |
I will take a look and push a PR for adjusting this! |
@joerg-nafcom so initially the data is added in correctly, and then upon updating it we get the ""e"? |
I hope that the title is the only place where this is needed since that is the only place where I allowed for html special chars. @joerg-nafcom please let me know if that will suffice, we are trying to ensure that there is some level of security so that html code cannot be executed by the browser accidentally. |
This was fixed due to great work by @ryangurn! |
Hi Ryan,
exactly!
I am sorry I didn't reply to this in time. I saw the notifcations but am super busy with Gamescom livestream this weekend.
Glad you've got this fixed, thank you! :)
Cheers!
Joerg
…On Sun, 30 Aug 2020 07:39:44 +0200, Ryan Gurnick ***@***.***> wrote:
@joerg-nafcom so initially the data is added in correctly, and then upon updating it we get the ""e"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
--
Using Opera's revolutionary e-mail client: https://en.wikipedia.org/wiki/Opera_Mail
--------------------------------------------------------------------------------------------
--------------------------------------------------------------------------------------------
DON'T SEND FILES BIGGER THAN 4 MB! THANKS!
(send to joerg.droege@gmail.com instead).
--------------------------------------------------------------------------------------------
Want to send to my iPhone? Use nafcom@games.com - thanks!
--------------------------------------------------------------------------------------------
|
@joerg-nafcom my pleasure, I will most likely be commenting on your other bug reports to get those squashed also. It was a pleasure. |
System Information
Podcast Generator Version: 3.1
Webserver: Apache
PHP Version:7.3
When this:
Is edited in Admin mode and I am simply hitting "Save" without changing anything, it defaces to this:
The text was updated successfully, but these errors were encountered: