Skip to content
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

Update A09_2021-Security_Logging_and_Monitoring_Failures.md #703

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fcerullo-cycubix
Copy link

Hi

While delivering a training, a student pointed out an improvement to the description of the A9 issue.

Hi

While delivering a training, a student pointed out an improvement to the description of the A9 issue.
@tobinbrooke
Copy link

Reasoning for putting more emphasis on log encoding as a means of preventing log stuffing:

  • By their very design logs are often able to be written to by un-authenticated users (logging failed login attempts)
  • Cannot rely on the output format of the log being unknown
  • Many systems still use a text/smi-structured log format
  • By crafting authentication payloads or other requests log entries that may emulate multiple entries can be crafted.
  • Fake entries could be created to trigger erronous IoA
  • By stuffing the logs this occurs before any signing and therefore breaks non-repudiation (make the log entries tell the desired story)
  • The OWASP top 10 item covers the log encoding in the "How to Prevent" section but this needs more emphasis such as in the description
  • Many systems can be exploited via un-authenticated or authenticated users (ease)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants