Container cleanup #266
Merged
Container cleanup #266
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 7 commits
July 13, 2021 09:27
…changed dummy value in nt_vars
msimerson
approved these changes
Jul 20, 2021
* Makefile.PL requirements changed from mysql to MariaDB * Updated Dockerfile to debian 10 with mariadb libraries Co-authored-by: Gerhard Steinbeis <[email protected]>
* Listen 443 is already present in /etc/apache2/sites-enabled/000-default.conf and causes issues (NicTool#263) * Improved nt_export to allow MariaDB dsn (NicTool#264) Co-authored-by: Gerhard Steinbeis <[email protected]>
…changed dummy value in nt_vars
…to container-cleanup
Contributor
Author
|
It seems there was a git conflict with the #265 which I have fixed now as it seems. Thanks for merging and approving the PR's so fast. :) |
Merged
msimerson
added a commit
that referenced
this pull request
Oct 3, 2025
* New Features
* allow A records in in-addr.arpa #230
* import
* tinydns type 'S' records #246
* bind: add CAA imports #225
* client: add SSHFP algos 4 & 6 #303
* export/BIND: add sort order #270
* Fixes
* bump DB version to 2.35 on new install (was 2.27)
* sql
* add quotes to rank (now a MySQL keyword)
* don't save empty strings for INTs #268
* zone.last_publish: TIMESTAMP DEFAULT 0 -> DATETIME DEFAULT NULL #249
* SRV: always octal_escape numbers correctly #302
* CAA: remove any quotes added to domain name #302
* update MANIFEST path #302
* NAPTR: do not export/encode quotes #302
* NAPTR: tinydns export fix #297
* URI: tinydns export needs target escaped #296
* export: store last ts separate #247
* client: fix user password change #241
* Updates & Changes
* client: add help for CAA records #219
* client: require NAPTR replacement #221
* client: only send cookie when secure #181
* zr/sanity: don't allow spaces in names #238
* import/bind: strip trailing dot from zone #248
* import/bind: report reason on rejected records #178
* import/tinydns:
* append trailing . to SOA mname and rname if missing #166
* anchor record type detection for case lowering #165
* add imports for type 3 & 6 (IPv6 patch) #167
* handle ascii and binary in AAAA rdata #163
* improved TXT & SPF imports #171
* disable SPF records by default (flip DB flag to enable) #168
* expand UTF support to cover all UTF8 characters (requires MySQL 5.5+)
* change table encoding from utf8 (3 bits) to utf8mb4 (4 bits) #170
* Export/BIND: export CAA correctly #193
* Export/BIND/dyn: fixes for ZR updates #199, #306
* Export/Dyn: more robust handling of zone deletions, that could
result in a new zone with the same name getting unpublished.
* Exports: split long SPF records into 255 byte strings #173
* Exports: run postflight after no-change export
* session: prune pass_salt from result hash
* session: omit sessions from deleted groups
* test: delete tests already migrated to nt3
* zone2nic: match domain w/o case #254
* improved sql/create_tables.pl #190
* docker: container updates #300, #269, #267, #266, #265, #264, #263, #262, #260
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Cleanup of the container setup
Issues with the current Docker setup
Changes proposed in this pull request:
During the build of the container, NO configuration files are required. This allows for a container image containing no configuration data and can be uploaded to public container registries. The configurationa nd customisation is added while starting the container.
When the /etc/nictool/ volume is mounted into the container when started, ...
This setup is more in line with the container philosophy and allows a more flexible update / change of settings. now changing configuration settings are simple. Change the configuration files mounted into /etc/nictool/ and restart the container. The same for the tls certificates, update them and restart the container.
ToDo:
Checklist: