mirror of https://github.com/gophish/gophish
e3352f481e
Initial commit of SSRF mitigations. This fixes #1908 by creating a *net.Dialer which restricts outbound connections to only allowed IP ranges. This implementation is based on the blog post at https://www.agwa.name/blog/post/preventing_server_side_request_forgery_in_golang To keep things backwards compatible, by default we'll only block connections to 169.254.169.254, the link-local IP address commonly used in cloud environments to retrieve metadata about the running instance. For other internal addresses (e.g. localhost or RFC 1918 addresses), it's assumed that those are available to Gophish. To support more secure environments, we introduce the `allowed_internal_hosts` configuration option where an admin can set one or more IP ranges in CIDR format. If addresses are specified here, then all internal connections will be blocked except to these hosts. There are various bits about this approach I don't really like. For example, since various packages all need this functionality, I had to make the RestrictedDialer a global singleton rather than a dependency off of, say, the admin server. Additionally, since webhooks are implemented via a singleton, I had to introduce a new function, `SetTransport`. Finally, I had to make an update in the gomail package to support a custom net.Dialer. |
||
|---|---|---|
| .. | ||
| attachment.go | ||
| campaign.go | ||
| campaign_test.go | ||
| doc.go | ||
| email_request.go | ||
| email_request_test.go | ||
| group.go | ||
| group_test.go | ||
| imap.go | ||
| maillog.go | ||
| maillog_test.go | ||
| models.go | ||
| models_test.go | ||
| page.go | ||
| page_test.go | ||
| rbac.go | ||
| rbac_test.go | ||
| result.go | ||
| result_test.go | ||
| smtp.go | ||
| smtp_test.go | ||
| template.go | ||
| template_context.go | ||
| template_context_test.go | ||
| user.go | ||
| user_test.go | ||
| webhook.go | ||