-
-
Notifications
You must be signed in to change notification settings - Fork 402
Unable to access a user's post PHPBB 3.3.3 #2150
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
Comments
I mention you directly, because I know you are the right person, when you can take a look, any info ask for it @azurit |
Hi @diegaless, thank your for your detailed report. I'm afraid we need more information to investigate this issue. You wrote the request is It would be good to see the whole request - can you share with us a |
sorry, rule 200004 was placed on me, it did not belong to this false positive, I have already edited the original message. The request is effectively a get, it did not make sense that it was a multipart, otherwise I would have specified it, I hope that I can continue working with this |
@diegaless Hi friend! Can you try this?
|
You are the best @azurit hahaha! It works! Thanks for everything, awesome work |
He's a wizard. :) But the question is whether this is a generic FP we ought to avoid in the future or something that has to be accepted. Looking at the payload containing Postgresql, I'm not surprised this was hit by a rule... |
For if it helps, this is why I jump to the rule specifically (I specifically think that the beginning of the sentence):
|
I think this can be integrated into exclusion package - it was matched on Edit: Also, rule 951240 is for PostgreSQL and phpBB is able to run only on MySQL / MariaDB. |
Good comment on 951240. Can you limit it to the RE to the very URI where the forum post is concerned? Ideally combined with GET method. |
phpbb supports postgre and more dbs: Info:phpbb db info |
Oh, sorry, didn't know that! Anyway, combined with GET method, i think it can be integrated anyway. @dune73 what do you think? |
Following CVE-2021-35638, I'm very weary of rule exclusions. I think it really depends on the PHPBB URIs. |
URI is exactly what's in the rule above: |
OK. Agreed. Than I think a PR should be written. (Sorry for not responding earlier) |
Probably |
I keep the original post, there is no match with 'ERROR' later on, but the word 'errores' is mentioned later, I don't know if that results in a submatch that breaks the rule. If I have understood correctly the 'ER' that you expose. If it is helpful I can provide the text of the original post |
@diegaless Can you, please, provide the text of the post? Thank you. |
Any update here @diegaless? |
Sorry for the delay and thank you for your work. Let me know once you have obtained the text of the post to edit and delete those parts of my comment. Excerpt from the text that I believe was the cause of the rule jump
Full Post:
|
Here is the exact match against pattern Do we have any examples of PostgreSQL error messages matching pattern |
This is a great example input. I have no experience with PostgreSQL, I briefly installed PostgreSQL and tried the basic example. Some errors that I got out of it are:
So it seems Maybe we should search for And maybe we should also have a separate rule that blocks anything with |
@azurit: this issue is assigned to you. Are you still interested to provide a fix? I was a bit weary first, but I think all we need is now on the table. What do you think? |
@dune73: Yes, i am. |
Cool. Is there anything you need, or are you good to go? |
I think there's everything. |
Cool. Thank you for the confirmation. |
@lifeforms I'm thinking about how to integrate check for |
@lifeforms ping :) |
I think SQLSTATE is so rare that my feeling is we won't get many false positives. |
This is solved via #2313 |
Uh oh!
There was an error while loading. Please reload this page.
Description
Unable to access a user's post (Forbidden), however he was able to post it successfully
Audit Logs / Triggered Rule Numbers
GET /viewtopic.php?f=55&t=1104&p=1705
Triggered rules:
\x0ao\x09Formaci\xc3\xb3n orientada \xc3\xbanica y exclusivamente a comandos de Cisco en PAR, el 80% del curso fueron comandos de Cisco, incluidos ex\xc3\xa1menes y el resto, gracias a dios subnetting, pero\xe2\x80\xa6 No se vio NADA de IPv6, nada \xc3\xbatil digo. A ver cuantos compa\xc3\xb1eros de curso saben las reglas de abreviaci\xc3\xb3n de ceros de una direcci\xc..."]
There were rules that were repeated and eliminated by synthesizing.
Your Environment
Confirmation
[x] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.
The text was updated successfully, but these errors were encountered: