User tests: Successful: Unsuccessful:
Pull Request resolves # .
Removed changes done with #48030
Code review.
Please select:
Documentation link for guide.joomla.org:
No documentation changes for guide.joomla.org needed
Pull Request link for manual.joomla.org:
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
| Category | ⇒ | Administration com_content Language & Strings Front End |
@brianteeman The code uses application secret stored in configuration.php for secure token creation and validation. However, that secret is not really secret. We use that secret to generate other data which is public viewable such as asset version, form token..., so in theory, that secret could be leaked. In case it is leaked, someone will have access to unauthenticated articles.
According to AI, it will take billion years at the moment to leak our 16 characters secret, so in practical, it's very unlikely this will happen but maintainers decided it could be potential security issue and decided to revert the PR.
yes i know that but whyv revert instead of fix - tha's the question
| Labels |
Added:
Language Change
PR-6.2-dev
|
||
Maintainers decided to revert PR 48030 for security reasons - this PR does this cleanly.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2026-08-31 18:31:07 |
| Closed_By | ⇒ | MacJoom |
Thank you!
@joomdonation, @brianteeman, @MacJoom, @drmenzelit If we won't fix the issue in the reverted PR, then why don't we go with the other one? It fixes the same issue but in another way other than the configuration.php secret token, we can also improve it. This is in case we won't fix the problem in the reverted PR.
@drmenzelit, i can switch the branch to 6.3 but need to know which approach we will take?
Will we fix the issue in the reverted PR or use the other one?
Discussed this in Maintainers (2.9.2026): We might think about a way to bypass ACL in a structured and secure way. So we are for a concept first and then follow with an implementation.
I made a preview unpublished article plugin a year ago. While it probably doesn't meet all your criteria for integrating into core, a few things worth considering:
Why?