Skip to content

Introduce NUT "authconf" file support [#3329, #3411]#3435

Open
jimklimov wants to merge 163 commits into
networkupstools:masterfrom
jimklimov:issue-3329
Open

Introduce NUT "authconf" file support [#3329, #3411]#3435
jimklimov wants to merge 163 commits into
networkupstools:masterfrom
jimklimov:issue-3329

Conversation

@jimklimov

@jimklimov jimklimov commented May 11, 2026

Copy link
Copy Markdown
Member

Starting with a PoC from AI, slightly modified (20%?) in review, following the spec requested in the GitHub issue #3329, as a stepping stone for further work. Also address parts of issue #3411.

Closes: #3503

Let CI loose on this iteration that passes locally...

TODO:

UPDATE: During work on this, it was found that NSS CERTHOST support was actually broken, and only worked for certificates whose nicknames matched the host name, not the "My nut server" nickname as the example claimed for years. Fixed now.

UPDATE: During work on this, it was found that it may be troublesome for a single client to connect to multiple NUT data servers which would require different CA trust stores and/or client self-identification by certificates. To be investigated under #3494, not in scope here.

@jimklimov jimklimov added this to the 2.8.6 milestone May 11, 2026
@jimklimov jimklimov added enhancement SSL/NSS Issues and PRs about SSL, TLS and other crypto-related matters AI For good or bad, machine tools are upon us. Humans are still the responsible ones. labels May 11, 2026
@github-actions

github-actions Bot commented May 11, 2026

Copy link
Copy Markdown

A ZIP file with standard source tarball and another tarball with pre-built docs for commit 9688c11 is temporarily available: NUT-tarballs-PR-3435.zip.

@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4693-master completed (commit 049a4d4c36 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4694-master completed (commit c8f40b4384 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4696-master completed (commit 25660e3752 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4697-master completed (commit 9be8443368 by @jimklimov)

Comment thread clients/authconf.c Fixed
Comment thread clients/authconf.c Fixed
Comment thread tests/test_authconf.c Fixed
Comment thread tests/test_authconf.c Fixed
Comment thread tests/test_authconf.c Fixed
@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Comment thread clients/authconf.c Dismissed
Comment thread clients/authconf.c Fixed
Comment thread clients/authconf.c Fixed
@jimklimov jimklimov force-pushed the issue-3329 branch 2 times, most recently from 73a7249 to ed6f549 Compare May 12, 2026 14:05
@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4700-master completed (commit 8a8a16fc63 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4701-master completed (commit 40e1ff94b4 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Comment thread clients/authconf.c Fixed
…_NUT_CAPATH (nss vs openssl backend) [networkupstools#3329, networkupstools#1711]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…_NUT_CERTHOST_NAME [networkupstools#3329, networkupstools#1711]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

…nt.pl, tests/cpputest-client.cpp: only set "NUT_SSL=True" from AuthConf NUT_FORCESSL if no value is passed as envvar [networkupstools#3329, networkupstools#1711]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…en if skipping other settings due to nutauth.conf [networkupstools#3329, networkupstools#1711]

May neuter later in the method (e.g. perl lacking SSL module).

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…nt.py.in: avoid Python 3 formatting strings [networkupstools#3229, networkupstools#1711]

There is no reason for module to not work for older OS installations.

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
… names after creation (fix IPv6 address handling) [networkupstools#3229, networkupstools#1711]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
…ly [networkupstools#3329]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
… nutauth-openssl-perl-win.conf [networkupstools#3329, networkupstools#1711]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
… a list, not vector [networkupstools#3329]

Memory-safer to extend and manipulate.

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4904-master completed (commit 81462e43ab by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

…TVERIFY also in nutauth.conf [networkupstools#1711]

Signed-off-by: Jim Klimov <jimklimov@gmail.com>
@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4905-master completed (commit b2bda9adb4 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4907-master completed (commit e397a14e21 by @jimklimov)

@AppVeyorBot

Copy link
Copy Markdown

…ackends [networkupstools#3331]

Signed-off-by: Jim Klimov <jimklimov+nut@gmail.com>
Comment thread clients/upsclient.c Dismissed
@AppVeyorBot

Copy link
Copy Markdown

Build nut 2.8.5.4915-master completed (commit c6c0f87401 by @jimklimov)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AI For good or bad, machine tools are upon us. Humans are still the responsible ones. enhancement SSL/NSS Issues and PRs about SSL, TLS and other crypto-related matters

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NUT authconf: make sure bracketed IPv6 addresses are parsed properly

3 participants