Skip to content

Patching Clear-text logging based on security checks - #193

Merged
isivaselvan merged 2 commits into
next-1.3.1from
fix/sec-vul
Jul 28, 2026
Merged

Patching Clear-text logging based on security checks#193
isivaselvan merged 2 commits into
next-1.3.1from
fix/sec-vul

Conversation

@isivaselvan

@isivaselvan isivaselvan commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Changes made

  • example file of oauth-client to skip printing the token id details.
  • http file log will capture the redacted url instead of logging full URLs.
  • Ruff check at Pyproject file to skip the markdown docs validation

Description

Testing plan

External links

Output from tests

Including output from tests may require access to a TFE instance. Ignore this section if you have no environment to test against.

Rollback Plan

Changes to Security Controls

PCI review checklist

  • I have documented a clear reason for, and description of, the change I am making.

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've documented the impact of any changes to security controls.

    Examples of changes to security controls include using new access control methods, adding or removing logging pipelines, etc.

If you have any questions, please contact your direct supervisor, GRC (#team-grc), or the PCI working group (#proj-pci-reboot). You can also find more information at PCI Compliance.

…ken details, http file log will capture the redacted url instead of logging full URLs
@isivaselvan
isivaselvan requested a review from a team as a code owner July 28, 2026 05:27
@isivaselvan
isivaselvan changed the base branch from main to next-1.3.1 July 28, 2026 06:53
@isivaselvan isivaselvan changed the title fix(sec-vul): Updated the example file of oauth-client to skip the to… Patching Clear-text logging based on security checks Jul 28, 2026
@isivaselvan
isivaselvan merged commit 8a1cbc2 into next-1.3.1 Jul 28, 2026
11 checks passed
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.

1 participant