Ed Tanous | 8041f31 | 2017-04-03 09:47:01 -0700 | [diff] [blame] | 1 | #pragma once |
| 2 | |
Ed Tanous | c94ad49 | 2019-10-10 15:39:33 -0700 | [diff] [blame] | 3 | #include <http_request.h> |
| 4 | #include <http_response.h> |
Ed Tanous | 8041f31 | 2017-04-03 09:47:01 -0700 | [diff] [blame] | 5 | |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 6 | namespace crow |
| 7 | { |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 8 | struct SecurityHeadersMiddleware |
| 9 | { |
| 10 | struct Context |
Gunnar Mills | 1214b7e | 2020-06-04 10:11:30 -0500 | [diff] [blame] | 11 | {}; |
Ed Tanous | 8041f31 | 2017-04-03 09:47:01 -0700 | [diff] [blame] | 12 | |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 13 | void beforeHandle(crow::Request& req, Response& res, Context& ctx) |
| 14 | { |
Ed Tanous | fd828ba | 2018-08-09 10:58:08 -0700 | [diff] [blame] | 15 | #ifdef BMCWEB_INSECURE_DISABLE_XSS_PREVENTION |
Ed Tanous | b41187f | 2019-10-24 16:30:02 -0700 | [diff] [blame^] | 16 | if (boost::beast::http::verb::options == req.method()) |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 17 | { |
| 18 | res.end(); |
| 19 | } |
| 20 | #endif |
Ed Tanous | fd828ba | 2018-08-09 10:58:08 -0700 | [diff] [blame] | 21 | } |
Ed Tanous | 8041f31 | 2017-04-03 09:47:01 -0700 | [diff] [blame] | 22 | |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 23 | void afterHandle(Request& req, Response& res, Context& ctx) |
| 24 | { |
| 25 | /* |
| 26 | TODO(ed) these should really check content types. for example, |
| 27 | X-UA-Compatible header doesn't make sense when retrieving a JSON or |
| 28 | javascript file. It doesn't hurt anything, it's just ugly. |
| 29 | */ |
Ed Tanous | 750ceae | 2018-11-30 15:02:22 -0800 | [diff] [blame] | 30 | using bf = boost::beast::http::field; |
| 31 | res.addHeader(bf::strict_transport_security, "max-age=31536000; " |
| 32 | "includeSubdomains; " |
| 33 | "preload"); |
| 34 | res.addHeader(bf::x_frame_options, "DENY"); |
| 35 | |
| 36 | res.addHeader(bf::pragma, "no-cache"); |
| 37 | res.addHeader(bf::cache_control, "no-Store,no-Cache"); |
Ed Tanous | 3eb2f35 | 2018-12-20 12:30:45 -0800 | [diff] [blame] | 38 | |
Ed Tanous | a3268f9 | 2019-09-23 14:06:03 -0700 | [diff] [blame] | 39 | res.addHeader("X-XSS-Protection", "1; " |
| 40 | "mode=block"); |
| 41 | res.addHeader("X-Content-Type-Options", "nosniff"); |
| 42 | |
| 43 | #ifndef BMCWEB_INSECURE_DISABLE_XSS_PREVENTION |
Ed Tanous | e6de21a | 2019-08-21 12:50:42 -0700 | [diff] [blame] | 44 | res.addHeader("Content-Security-Policy", "default-src 'none'; " |
| 45 | "img-src 'self' data:; " |
| 46 | "font-src 'self'; " |
| 47 | "style-src 'self'; " |
| 48 | "script-src 'self'; " |
| 49 | "connect-src 'self' wss:"); |
| 50 | // The KVM currently needs to load images from base64 encoded |
| 51 | // strings. img-src 'self' data: is used to allow that. |
Ed Tanous | 3eb2f35 | 2018-12-20 12:30:45 -0800 | [diff] [blame] | 52 | // https://stackoverflow.com/questions/18447970/content-security-policy-data-not-working-for-base64-images-in-chrome-28 |
Ed Tanous | e6de21a | 2019-08-21 12:50:42 -0700 | [diff] [blame] | 53 | |
Ed Tanous | a3268f9 | 2019-09-23 14:06:03 -0700 | [diff] [blame] | 54 | #else |
| 55 | // If XSS is disabled, we need to allow loading from addresses other |
| 56 | // than self, as the BMC will be hosted elsewhere. |
| 57 | res.addHeader("Content-Security-Policy", "default-src 'none'; " |
| 58 | "img-src *; " |
| 59 | "font-src *; " |
| 60 | "style-src *; " |
| 61 | "script-src *; " |
| 62 | "connect-src *"); |
Ed Tanous | fd828ba | 2018-08-09 10:58:08 -0700 | [diff] [blame] | 63 | |
Ed Tanous | a3268f9 | 2019-09-23 14:06:03 -0700 | [diff] [blame] | 64 | const std::string_view origin = req.getHeaderValue("Origin"); |
| 65 | res.addHeader(bf::access_control_allow_origin, origin); |
Ed Tanous | 750ceae | 2018-11-30 15:02:22 -0800 | [diff] [blame] | 66 | res.addHeader(bf::access_control_allow_methods, "GET, " |
| 67 | "POST, " |
| 68 | "PUT, " |
Ed Tanous | da7f41e | 2018-12-10 13:36:40 -0800 | [diff] [blame] | 69 | "PATCH, " |
| 70 | "DELETE"); |
Ed Tanous | 750ceae | 2018-11-30 15:02:22 -0800 | [diff] [blame] | 71 | res.addHeader(bf::access_control_allow_credentials, "true"); |
| 72 | res.addHeader(bf::access_control_allow_headers, "Origin, " |
| 73 | "Content-Type, " |
| 74 | "Accept, " |
| 75 | "Cookie, " |
| 76 | "X-XSRF-TOKEN"); |
Ed Tanous | fd828ba | 2018-08-09 10:58:08 -0700 | [diff] [blame] | 77 | |
| 78 | #endif |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 79 | } |
Ed Tanous | 8041f31 | 2017-04-03 09:47:01 -0700 | [diff] [blame] | 80 | }; |
Ed Tanous | 1abe55e | 2018-09-05 08:30:59 -0700 | [diff] [blame] | 81 | } // namespace crow |