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

Strip version disclosures from response headers #32

Open
jfrank-nih opened this issue Jun 29, 2022 · 4 comments
Open

Strip version disclosures from response headers #32

jfrank-nih opened this issue Jun 29, 2022 · 4 comments

Comments

@jfrank-nih
Copy link
Member

jfrank-nih commented Jun 29, 2022

Response headers for certain pages contain information about the IIS and ASP.NET versions used. NCI recommendations are to suppress version information in responses.

Issue

URL: https://www-test-acsf.cancer.gov/Common/PopUps/popHelp.html
Response Headers: server contains the IIS version

URL: https://www-test-acsf.cancer.gov/Common/PopUps/popDefinition.aspx?id=CDR0000045849&language=English&version=Patient
Response Headers: server contains the IIS version and x-aspnet-version contains software version

Remedy

Remove the offending headers.

@jfrank-nih
Copy link
Member Author

@blairlearn, we could bug the hosting team, or... everything passes through Akamai from the origin, correct? In which case we could strip out headers there.

@blairlearn
Copy link
Contributor

Unfortunately, prior to IIS 10, there's no ability to remove the server header.
Possibly the x-aspnet-version header.

@jfrank-nih
Copy link
Member Author

Fair enough. But could we remove with Akamai?

@blairlearn
Copy link
Contributor

No, this isn't a problem in CGDP, it shows up there because of how things are mapped through Akamai. The correct fix is to address it in dynamic services. (Which is where I'll be moving this ticket momentarily.)

@blairlearn blairlearn transferred this issue from NCIOCPL/wcms-cde Jun 30, 2022
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

No branches or pull requests

2 participants