ghsa-8274-h5jp-97vr
Vulnerability from github
Impact
Applications that use Diactoros, and are either not behind a proxy, or can be accessed via untrusted proxies, can potentially have the host, protocol, and/or port of a Laminas\Diactoros\Uri
instance associated with the incoming server request modified to reflect values from X-Forwarded-*
headers. Such changes can potentially lead to XSS attacks (if a fully-qualified URL is used in links) and/or URL poisoning.
Patches
Any version after 2.11.0.
Starting in laminas/laminas-diactoros 2.11.1, we have added Laminas\Diactoros\ServerRequestFilter\FilterServerRequestInterface
, which defines the single method __invoke(Psr\Http\Message\ServerRequestInterface $request): Psr\Http\Message\ServerRequestInterface
. Filters implementing this interface allow modifying and returning a generated ServerRequest
.
The primary use case of the interface is to allow modifying the generated URI based on the presence of headers such as X-Forwarded-Host
. When operating behind a reverse proxy, the Host
header is often rewritten to the name of the node to which the request is being forwarded, and an X-Forwarded-Host
header is generated with the original Host
value to allow the server to determine the original host the request was intended for. (We have always examined the X-Forwarded-Proto
header; as of Diactoros 2.11.1, we also examine the X-Forwarded-Port
header.) To accommodate this use case, we created Laminas\Diactoros\ServerRequestFilter\FilterUsingXForwardedHeaders.
Due to potential security issues, it is generally best to only accept these headers if you trust the reverse proxy that has initiated the request.
(This value is found in $_SERVER['REMOTE_ADDR']
, which is present as $request->getServerParams()['REMOTE_ADDR']
within PSR-7 implementations.) FilterUsingXForwardedHeaders
provides named constructors to allow you to trust these headers from any source (which has been the default behavior of Diactoros since the beginning), or to specify specific IP addresses or CIDR subnets to trust, along with which headers are trusted.
Laminas\Diactoros\ServerRequestFactory::fromGlobals()
was updated to accept a FilterServerRequestInterface
as an additional, optional argument. Since the X-Forwarded-*
headers do have valid use cases, particularly in clustered environments using a load balancer, to prevent backwards compatibility breaks, if no filter is provided, we generate an instance via FilterUsingXForwardedHeaders::trustReservedSubnets()
, which generates an instance marked to trust only proxies on private subnets.
Workarounds
Infrastructure or DevOps can configure web servers to reject X-Forwarded-*
headers at the web server level.
Users of laminas/laminas-diactoros can make use of the Laminas\Diactoros\RequestFilter\RequestFilterInterface
functionality in order to either (a) disable usage of the X-Forwarded-*
headers entirely, (b) opt-in to it, or (c) opt-in to the usage for configured proxy servers.
References
For more information
If you have any questions or comments about this advisory:
- Open an issue in laminas/laminas-diactoros
- Email us
{ "affected": [ { "package": { "ecosystem": "Packagist", "name": "laminas/laminas-diactoros" }, "ranges": [ { "events": [ { "introduced": "0" }, { "fixed": "2.11.1" } ], "type": "ECOSYSTEM" } ] } ], "aliases": [ "CVE-2022-31109" ], "database_specific": { "cwe_ids": [ "CWE-79" ], "github_reviewed": true, "github_reviewed_at": "2022-07-27T22:05:18Z", "nvd_published_at": "2022-08-01T17:15:00Z", "severity": "MODERATE" }, "details": "### Impact\n\nApplications that use Diactoros, and are either not behind a proxy, or can be accessed via untrusted proxies, can potentially have the host, protocol, and/or port of a `Laminas\\Diactoros\\Uri` instance associated with the incoming server request modified to reflect values from `X-Forwarded-*` headers. Such changes can potentially lead to XSS attacks (if a fully-qualified URL is used in links) and/or URL poisoning.\n\n### Patches\n\nAny version after 2.11.0.\n\nStarting in laminas/laminas-diactoros 2.11.1, we have added `Laminas\\Diactoros\\ServerRequestFilter\\FilterServerRequestInterface`, which defines the single method `__invoke(Psr\\Http\\Message\\ServerRequestInterface $request): Psr\\Http\\Message\\ServerRequestInterface`. Filters implementing this interface allow modifying and returning a generated `ServerRequest`.\n\nThe primary use case of the interface is to allow modifying the generated URI based on the presence of headers such as `X-Forwarded-Host`. When operating behind a reverse proxy, the `Host` header is often rewritten to the name of the node to which the request is being forwarded, and an `X-Forwarded-Host` header is generated with the original `Host` value to allow the server to determine the original host the request was intended for. (We have always examined the `X-Forwarded-Proto` header; as of Diactoros 2.11.1, we also examine the `X-Forwarded-Port` header.) To accommodate this use case, we created Laminas\\Diactoros\\ServerRequestFilter\\FilterUsingXForwardedHeaders.\n\nDue to potential security issues, it is generally best to only accept these headers if you trust the reverse proxy that has initiated the request.\n(This value is found in `$_SERVER[\u0027REMOTE_ADDR\u0027]`, which is present as `$request-\u003egetServerParams()[\u0027REMOTE_ADDR\u0027]` within PSR-7 implementations.) `FilterUsingXForwardedHeaders` provides named constructors to allow you to trust these headers from any source (which has been the default behavior of Diactoros since the beginning), or to specify specific IP addresses or CIDR subnets to trust, along with which headers are trusted.\n\n`Laminas\\Diactoros\\ServerRequestFactory::fromGlobals()` was updated to accept a `FilterServerRequestInterface` as an additional, optional argument. Since the `X-Forwarded-*` headers do have valid use cases, particularly in clustered environments using a load balancer, to prevent backwards compatibility breaks, if no filter is provided, we generate an instance via `FilterUsingXForwardedHeaders::trustReservedSubnets()`, which generates an instance marked to trust only proxies on private subnets.\n\n### Workarounds\n\nInfrastructure or DevOps can configure web servers to reject `X-Forwarded-*` headers at the web server level.\n\nUsers of laminas/laminas-diactoros can make use of the `Laminas\\Diactoros\\RequestFilter\\RequestFilterInterface` functionality in order to either (a) disable usage of the `X-Forwarded-*` headers entirely, (b) opt-in to it, or (c) opt-in to the usage for configured proxy servers.\n\n### References\n\n- [HTTP Host Header Attacks](https://portswigger.net/web-security/host-header)\n\n### For more information\n\nIf you have any questions or comments about this advisory:\n\n- Open an issue in [laminas/laminas-diactoros](https://github.com/laminas/laminas-diactoros/)\n- [Email us](mailto:security@getlaminas.org)", "id": "GHSA-8274-h5jp-97vr", "modified": "2022-08-10T23:52:25Z", "published": "2022-07-27T22:05:18Z", "references": [ { "type": "WEB", "url": "https://github.com/laminas/laminas-diactoros/security/advisories/GHSA-8274-h5jp-97vr" }, { "type": "ADVISORY", "url": "https://nvd.nist.gov/vuln/detail/CVE-2022-31109" }, { "type": "WEB", "url": "https://github.com/laminas/laminas-diactoros/commit/25b11d422c2e5dad868f68619888763b30f91e2d" }, { "type": "WEB", "url": "https://github.com/FriendsOfPHP/security-advisories/blob/master/laminas/laminas-diactoros/CVE-2022-31109.yaml" }, { "type": "ADVISORY", "url": "https://github.com/advisories/GHSA-8274-h5jp-97vr" }, { "type": "PACKAGE", "url": "https://github.com/laminas/laminas-diactoros" }, { "type": "WEB", "url": "https://github.com/laminas/laminas-diactoros/releases/tag/2.11.1" }, { "type": "WEB", "url": "https://portswigger.net/web-security/host-header" } ], "schema_version": "1.4.0", "severity": [ { "score": "CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:L/I:L/A:N", "type": "CVSS_V3" } ], "summary": "Diactoros before 2.11.1 vulnerable to HTTP Host Header Attack" }
Sightings
Author | Source | Type | Date |
---|
Nomenclature
- Seen: The vulnerability was mentioned, discussed, or seen somewhere by the user.
- Confirmed: The vulnerability is confirmed from an analyst perspective.
- Exploited: This vulnerability was exploited and seen by the user reporting the sighting.
- Patched: This vulnerability was successfully patched by the user reporting the sighting.
- Not exploited: This vulnerability was not exploited or seen by the user reporting the sighting.
- Not confirmed: The user expresses doubt about the veracity of the vulnerability.
- Not patched: This vulnerability was not successfully patched by the user reporting the sighting.