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

x/vulndb: potential Go vuln in github.com/nats-io/nats-server #2132

Closed
1 task done
philpennock opened this issue Oct 19, 2023 · 1 comment
Closed
1 task done

x/vulndb: potential Go vuln in github.com/nats-io/nats-server #2132

philpennock opened this issue Oct 19, 2023 · 1 comment

Comments

@philpennock
Copy link

Acknowledgement

  • The maintainer(s) of the affected project have already been made aware of this vulnerability.

Description

(This advisory is canonically https://advisories.nats.io/CVE/secnote-2023-01.txt)
(Advisory written from perspective of the nats-server binary, but the server can be, and is, embedded in other applications)

Background

NATS.io is a high performance open source pub-sub distributed communication technology, built for the cloud, on-premise, IoT, and edge computing.

NATS users exist within accounts, and once using accounts, the old authorization block is not applicable.

Problem Description

Without any authorization rules in the nats-server, users can connect without authentication.

Before nats-server 2.2.0, all authentication and authorization rules for a nats-server lived in an "authorization" block, defining users. With nats-server 2.2.0 all users live inside accounts. When using the authorization block, whose syntax predates this, those users will be placed into the implicit global account, "$G". Users inside accounts go into the newer "accounts" block.

If an "accounts" block is defined, in simple deployment scenarios this is often used only to enable client access to the system account. When the only account added is the system account "$SYS", the nats-server would create an implicit user in "$G" and set it as the no_auth_user account, enabling the same "without authentication" logic as without any rules.

This preserved the ability to connect simply, and then add one authenticated login for system access.

But with an "authorization" block, this is wrong. Users exist in the global account, with login rules. And in simple testing, they might still connect fine without administrators seeing that authentication has been disabled.

The blind-spot on our part came from encouraging and documenting a switch to using only "accounts", instead of "authorization".

In the fixed versions, using an "authorization" block will inhibit the implicit creation of a "$G" user and setting it as the no_auth_user target. In unfixed versions, just creating a second account, with no users, will also inhibit this behavior.

Affected versions

NATS Server:

  • 2.2.0 up to and including 2.9.22 and 2.10.1
  • Fixed with nats-io/nats-server: 2.10.2 and backported to 2.9.23

Workarounds

In the "accounts" block, define a second non-system account, leave it empty.

accounts {
    SYS: {
        users: [
            { user: sysuser, password: makemeasandwich }
        ]
    }
    DUMMY: {}  # for security, before 2.10.2
}
system_account: SYS

Solution

Any one of these:

  1. Upgrade the NATS server to at least 2.10.2 (or 2.9.23)
  2. Or define a dummy account
  3. Or complete the migration of authorization entries to be inside a named account in the "accounts" block

Credits

Problem reported by Alex Herrington.
Addressed publicly in a GitHub Discussion prior to this advisory.

Affected Modules, Packages, Versions and Symbols

Module: github.com/nats-io/nats-server/v2
Package: github.com/nats-io/nats-server/v2/server
Versions:
  - Introduced: 2.2.0
  - Fixed: 2.10.2
  - Fixed: 2.9.23
Symbols:
  - ConfigureOptions
  - Options.ProcessConfigFile

CVE/GHSA ID

GHSA-fr2g-9hjm-wr23

Fix Commit or Pull Request

No response

References

Additional information

No response

@tatianab
Copy link
Contributor

Thanks for bringing this to our attention; this advisory is now published as GO-2023-2133, tracked in issue #2133

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