Skip to content

added missing url logging#1040

Merged
securestep9 merged 2 commits intoOWASP:masterfrom
securestep9:logging_add
Mar 18, 2025
Merged

added missing url logging#1040
securestep9 merged 2 commits intoOWASP:masterfrom
securestep9:logging_add

Conversation

@securestep9
Copy link
Collaborator

Proposed change

Your PR description goes here.
added missing url logging

Type of change

  • New core framework functionality
  • Bugfix (non-breaking change which fixes an issue)
  • Code refactoring without any functionality changes
  • New or existing module/payload change
  • Localization improvement
  • Dependency upgrade
  • Documentation improvement

Checklist

  • I've followed the contributing guidelines
  • I've run make pre-commit, it didn't generate any changes
  • I've run make test, all tests passed locally

@securestep9 securestep9 requested a review from arkid15r as a code owner March 10, 2025 00:38
arkid15r
arkid15r previously approved these changes Mar 10, 2025
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-authored-by: Arkadii Yakovets <2201626+arkid15r@users.noreply.github.com>
Signed-off-by: Sam Stepanyan <sam.stepanyan@owasp.org>
Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, let's merge it

@securestep9 securestep9 added this pull request to the merge queue Mar 18, 2025
Merged via the queue into OWASP:master with commit 6ad4ce0 Mar 18, 2025
17 checks passed
@securestep9 securestep9 deleted the logging_add branch March 18, 2025 00:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants