Skip to content

Conversation

@aknot242
Copy link
Collaborator

@aknot242 aknot242 commented Jan 5, 2026

Proposed changes

Checklist

Before sharing this pull request, I completed the following checklist:

Footnotes

  1. Potentially sensitive information includes personally identify information (PII), authentication credentials, and live URLs. Refer to the style guide for guidance about placeholder content.

@aknot242 aknot242 requested a review from a team as a code owner January 5, 2026 16:19
@github-actions github-actions bot added documentation Improvements or additions to documentation product/nim Issues related to NGINX Instance Manager product/dos Issues related to F5 DoS for NGINX labels Jan 5, 2026
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Deploy Preview will be available once build job completes!

Name Link
😎 Deploy Preview https://frontdoor-test-docs.nginx.com/previews/docs/1630/

Copy link
Contributor

@mjang mjang left a comment

Choose a reason for hiding this comment

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

Approved with one comment (I think you may have missed a couple of cases)

@mjang
Copy link
Contributor

mjang commented Jan 5, 2026

Wait for SME review before merging, @vadimkrishtalf5

@mjang
Copy link
Contributor

mjang commented Jan 6, 2026

Approved by @vadimkrishtalf5 in Slack, therefore merging

@mjang mjang merged commit 7588457 into main Jan 6, 2026
9 checks passed
@mjang mjang deleted the nim-remove-headers branch January 6, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation product/dos Issues related to F5 DoS for NGINX product/nim Issues related to NGINX Instance Manager

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants