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

OWASP 10 Security issues #10

Closed
gnewton opened this issue Aug 11, 2015 · 2 comments
Closed

OWASP 10 Security issues #10

gnewton opened this issue Aug 11, 2015 · 2 comments

Comments

@gnewton
Copy link

gnewton commented Aug 11, 2015

OWASP 10 https://www.owasp.org/index.php/Category:OWASP_Top_Ten_Project "The OWASP Top Ten represents a broad consensus about what the most critical web application security flaws are."

I applied a tool that tests for OWASP 10 vulnerabilities: https://www.owasp.org/index.php/OWASP_Zed_Attack_Proxy_Project

It found a couple, mostly medium level. They involve some headers not being set:

  • "X-Frame-Options header is not included in the HTTP response to protect against 'ClickJacking' attacks."
  • "Web Browser XSS Protection is not enabled, or is disabled by the configuration of the 'X-XSS-Protection' HTTP response header on the web server"
  • "The Anti-MIME-Sniffing header X-Content-Type-Options was not set to 'nosniff'. This allows older versions of Internet Explorer and Chrome to perform MIME-sniffing on the response body, potentially causing the response body to be interpreted and displayed as a content type other than the declared content type. Current (early 2014) and legacy versions of Firefox will use the declared content type (if one is set), rather than performing MIME-sniffing."

Is there any chance the gowut.Server interface could allow setting the underlying http.Response.Header? i.e. setting one or more headers that are sent with every response?

The report for the tool is here: https://drive.google.com/file/d/0B9uNae3afGInUnI2VEI3aENVZEk/view?usp=sharing

@icza
Copy link
Owner

icza commented Aug 11, 2015

This is very easily doable, although current implementation does not support it.

Will add support for this.

@icza
Copy link
Owner

icza commented Mar 7, 2016

Implemented adding custom headers.

New methods in Server: SetHeader() and Header().
You can use these to set headers that will be added to all responses.

Changes are committed to the gowut.dev project, will be available in the next release.

Commit:
icza/gowut.dev@12e5c40

Thanks for reporting this.

@icza icza closed this as completed Mar 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants