nginx default config has SSLv3 enabled, makes sites using default config options vulnerable to POODLE
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Nginx |
Medium
|
Thomas Ward | ||
| nginx (Ubuntu) |
Undecided
|
Unassigned | ||
| Precise |
Undecided
|
Unassigned | ||
| Trusty |
Undecided
|
Unassigned | ||
| Utopic |
Undecided
|
Unassigned |
Bug Description
The included `default` config file contains a commented-out section for SSL.
That SSL section has the SSLv3 parameter provided for `ssl_protocols`. This means that systems are vulnerable to SSLv3 and the POODLE vulnerability.
Can we remove that from the default section, even though it's commented out, so users don't use the insecure SSLv3 protocol anymore?
------
NGINX Project:
In the PPAs, this affects all versions of the package in both Stable and Mainline.
------
Ubuntu Project:
In Ubuntu, this affects the Precise, Trusty, and Utopic versions of the package.
This change was already made/committed in Debian Unstable.
summary: |
- nginx default config has SSLv3 enabled, makes things vulnerable to - POODLE + nginx default config has SSLv3 enabled, makes sites using default config + options vulnerable to POODLE |
Changed in nginx: | |
status: | New → Confirmed |
description: | updated |
description: | updated |
description: | updated |
description: | updated |
Changed in nginx: | |
assignee: | nobody → Thomas Ward (teward) |
tags: | added: poodle |
Thomas Ward (teward) wrote : | #1 |
Thomas Ward (teward) wrote : | #2 |
Thomas Ward (teward) wrote : | #3 |
Thomas Ward (teward) wrote : | #4 |
NOTE: I didn't know where to target this for Utopic, so I just targeted it to 'utopic'. Please change that if it is not valid.
Changed in nginx: | |
status: | Confirmed → In Progress |
The attachment "Precise Debdiff for Ubuntu" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.
[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]
tags: | added: patch |
Thomas Ward (teward) wrote : | #6 |
(nginx project)
Updated packages uploaded to the staging PPA, pending building.
Changed in nginx: | |
status: | In Progress → Fix Committed |
Changed in nginx (Ubuntu Utopic): | |
status: | New → Fix Released |
Thomas Ward (teward) wrote : | #7 |
Fixed in latest NGINX upload to the PPAs.
Changed in nginx: | |
status: | Fix Committed → Fix Released |
In discussion with mdeslaur on IRC, I'm attaching DebDiffs for Ubuntu in the off chance the release team wishes to push these changes.
A few extra details as to why this is extremely relevant to being pushed and updated: A lot of newbie users that we see in the NGINX IRC channel for support end up using the default configuration file(s) as a template/law for their sites, and we end up seeing them just uncomment the SSL portion and use it. That opens them up to the POODLE vulnerability.
Upstream, in Debian, this was revised per POODLE so newbies wouldn't be exposing themselves to vulnerability, hence the rationale for this perhaps being pushed/updated. Unfortunately, all versions of nginx (including Lucid, which is being ignored intentionally) are impacted by POODLE and SSLv3, so it's better to just remove the SSLv3 protocol from the ssl_protocols line, mainly to prevent newbies from making their systems open to attack.