Hide nginx server header from response #2161

RajeshPaul38 · 2021-08-18T12:19:25Z

New functionality includes tests
All buildkite tests pass
Full omnibus build and tests in buildkite pass
All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
PR title is a worthy inclusion in the CHANGELOG

Signed-off-by: Rajesh Paul <rajesh.paul@progress.com>

…stead of nginx

Signed-off-by: Rajesh Paul <rajesh.paul@progress.com>

tas50 · 2021-08-18T16:32:21Z

RajeshPaul38 added 3 commits August 17, 2021 21:41

added dependency openresty; removed dependency nginx

20eed20

removed commented code

cd2e71f

Signed-off-by: Rajesh Paul <rajesh.paul@progress.com>

changed inspec test to check for server header value should be nil in…

96285e0

…stead of nginx

RajeshPaul38 requested review from a team August 18, 2021 12:19

RajeshPaul38 requested a review from a team as a code owner August 18, 2021 12:19

RajeshPaul38 mentioned this pull request Aug 18, 2021

Make sure server headers do not report nginx #1978

Closed

RajeshPaul38 linked an issue Aug 18, 2021 that may be closed by this pull request

Make sure server headers do not report nginx #1978

Closed

RajeshPaul38 added 2 commits August 18, 2021 18:07

checking with nil test directive change

26f02da

typo fix

3359189

Signed-off-by: Rajesh Paul <rajesh.paul@progress.com>

RajeshPaul38 force-pushed the rajeshpaul38/hide-nginx-server-header-from-response branch from 24dec17 to 3359189 Compare August 18, 2021 12:54

saghoshprogress approved these changes Aug 18, 2021

View reviewed changes

tas50 merged commit 75e982b into master Aug 18, 2021

tas50 deleted the rajeshpaul38/hide-nginx-server-header-from-response branch August 18, 2021 16:32

Provide feedback