refactor: make AutoConf.URL implicit like AutoTLS

- change AutoConf.URL to *OptionalString with omitempty
- use WithDefault(DefaultAutoConfURL) for implicit defaults
- update migration to omit URL field for cleaner config
- remove explicit empty field validation

this makes AutoConf work like AutoTLS where defaults are
implicit unless explicitly overridden by the user.

cmd/ipfs/kubo/daemon.go

@@ -353,10 +353,6 @@ func daemonFunc(req *cmds.Request, re cmds.ResponseEmitter, env cmds.Environment
 
 	// Start background AutoConf updater if enabled
 	if cfg.AutoConf.Enabled.WithDefault(config.DefaultAutoConfEnabled) {
-		autoConfURL := cfg.AutoConf.URL
-		if autoConfURL == "" {
-			return fmt.Errorf("AutoConf is enabled but AutoConf.URL is empty - please provide a URL")
-		}
 		// Start autoconf client for background updates
 		client, err := config.GetAutoConfClient(cfg)
 		if err != nil {

config/autoconf.go

@@ -16,7 +16,7 @@ var log = logging.Logger("config")
 type AutoConf struct {
 	// URL is the HTTP(S) URL to fetch the autoconf.json from
 	// Default: see boxo/autoconf.MainnetAutoConfURL
-	URL string
+	URL *OptionalString `json:",omitempty"`
 
 	// Enabled determines whether to use autoconf
 	// Default: true

config/autoconf_client.go

@@ -43,10 +43,7 @@ func newAutoConfClient(cfg *Config) (*autoconf.Client, error) {
 	}
 
 	// Use default URL if not specified
-	url := cfg.AutoConf.URL
-	if url == "" {
-		url = DefaultAutoConfURL
-	}
+	url := cfg.AutoConf.URL.WithDefault(DefaultAutoConfURL)
 
 	// Build client options
 	options := []autoconf.Option{
@@ -69,13 +66,9 @@ func ValidateAutoConfWithRepo(cfg *Config, swarmKeyExists bool) error {
 		return validateAutoConfDisabled(cfg)
 	}
 
-	// AutoConf is enabled - validate URL is provided
-	if cfg.AutoConf.URL == "" {
-		return fmt.Errorf("AutoConf is enabled but AutoConf.URL is empty - please provide a URL")
-	}
-
 	// Check for private network with default mainnet URL
-	if swarmKeyExists && cfg.AutoConf.URL == DefaultAutoConfURL {
+	url := cfg.AutoConf.URL.WithDefault(DefaultAutoConfURL)
+	if swarmKeyExists && url == DefaultAutoConfURL {
 		return fmt.Errorf("AutoConf cannot use the default mainnet URL (%s) on a private network (swarm.key or LIBP2P_FORCE_PNET detected). Either disable AutoConf by setting AutoConf.Enabled=false, or configure AutoConf.URL to point to a configuration service specific to your private swarm", DefaultAutoConfURL)
 	}
 

config/autoconf_test.go

@@ -11,12 +11,12 @@ func TestAutoConfDefaults(t *testing.T) {
 	// Test that AutoConf has the correct default values
 	cfg := &Config{
 		AutoConf: AutoConf{
-			URL:     DefaultAutoConfURL,
+			URL:     NewOptionalString(DefaultAutoConfURL),
 			Enabled: True,
 		},
 	}
 
-	assert.Equal(t, DefaultAutoConfURL, cfg.AutoConf.URL)
+	assert.Equal(t, DefaultAutoConfURL, cfg.AutoConf.URL.WithDefault(DefaultAutoConfURL))
 	assert.True(t, cfg.AutoConf.Enabled.WithDefault(DefaultAutoConfEnabled))
 
 	// Test default refresh interval
@@ -63,7 +63,7 @@ func TestAutoConfProfile(t *testing.T) {
 	assert.True(t, cfg.AutoConf.Enabled.WithDefault(DefaultAutoConfEnabled))
 
 	// Check that URL was set
-	assert.Equal(t, DefaultAutoConfURL, cfg.AutoConf.URL)
+	assert.Equal(t, DefaultAutoConfURL, cfg.AutoConf.URL.WithDefault(DefaultAutoConfURL))
 }
 
 func TestInitWithAutoValues(t *testing.T) {
@@ -88,5 +88,5 @@ func TestInitWithAutoValues(t *testing.T) {
 
 	// Check that AutoConf is enabled with correct URL
 	assert.True(t, cfg.AutoConf.Enabled.WithDefault(DefaultAutoConfEnabled))
-	assert.Equal(t, DefaultAutoConfURL, cfg.AutoConf.URL)
+	assert.Equal(t, DefaultAutoConfURL, cfg.AutoConf.URL.WithDefault(DefaultAutoConfURL))
 }

config/init.go

@@ -76,10 +76,6 @@ func InitWithIdentity(identity Identity) (*Config, error) {
 			DownloadSources: []string{},
 			Keep:            "",
 		},
-		AutoConf: AutoConf{
-			URL:     DefaultAutoConfURL,
-			Enabled: True,
-		},
 		Routing: Routing{
 			DelegatedRouters: []string{AutoPlaceholder},
 		},

config/ipns.go

@@ -22,5 +22,5 @@ type Ipns struct {
 	UsePubsub Flag `json:",omitempty"`
 
 	// Simplified configuration for delegated IPNS publishers
-	DelegatedPublishers []string `json:",omitempty"`
+	DelegatedPublishers []string
 }

config/profile.go

@@ -87,6 +87,12 @@ is useful when using the daemon in test environments.`,
 			c.Bootstrap = []string{}
 			c.Discovery.MDNS.Enabled = false
 			c.AutoTLS.Enabled = False
+			c.AutoConf.Enabled = False
+
+			// Explicitly set autoconf-controlled fields to empty when autoconf is disabled
+			c.DNS.Resolvers = map[string]string{}
+			c.Routing.DelegatedRouters = []string{}
+			c.Ipns.DelegatedPublishers = []string{}
 			return nil
 		},
 	},
@@ -99,10 +105,8 @@ Inverse profile of the test profile.`,
 
 			// Use AutoConf system for bootstrap peers
 			c.Bootstrap = []string{AutoPlaceholder}
-			c.AutoConf.Enabled = True
-			if c.AutoConf.URL == "" {
-				c.AutoConf.URL = DefaultAutoConfURL
-			}
+			c.AutoConf.Enabled = Default
+			c.AutoConf.URL = nil // Clear URL to use implicit default
 
 			c.Swarm.DisableNatPortMap = false
 			c.Discovery.MDNS.Enabled = true
@@ -363,8 +367,8 @@ This profile requires AutoConf to be enabled and configured.`,
 			c.Routing.DelegatedRouters = []string{AutoPlaceholder}
 			c.Ipns.DelegatedPublishers = []string{AutoPlaceholder}
 			c.AutoConf.Enabled = True
-			if c.AutoConf.URL == "" {
-				c.AutoConf.URL = DefaultAutoConfURL
+			if c.AutoConf.URL == nil {
+				c.AutoConf.URL = NewOptionalString(DefaultAutoConfURL)
 			}
 			return nil
 		},

config/routing.go

@@ -46,7 +46,7 @@ type Routing struct {
 	IgnoreProviders []string `json:",omitempty"`
 
 	// Simplified configuration used by default when Routing.Type=auto|autoclient
-	DelegatedRouters []string `json:",omitempty"`
+	DelegatedRouters []string
 
 	// Advanced configuration used when Routing.Type=custom
 	Routers Routers `json:",omitempty"`

docs/config.md

@@ -661,12 +661,14 @@ Specifies the HTTP(S) URL from which to fetch the autoconf JSON. The endpoint sh
 
 The URL must serve a JSON document matching the AutoConf schema. Kubo validates all multiaddr and URL values before caching to ensure they are properly formatted.
 
+When not specified in the configuration, the default mainnet URL is used automatically.
+
 <a href="https://ipshipyard.com/"><img align="right" src="https://github.com/user-attachments/assets/39ed3504-bb71-47f6-9bf8-cb9a1698f272" /></a>
 
 > [!NOTE]
 > Public good autoconf manifest at `conf.ipfs-mainnet.org` is provided by the team at [Shipyard](https://ipshipyard.com).
 
-Default: `"https://example.com/autoconf.json"`
+Default: `"https://conf.ipfs-mainnet.org/autoconf.json"` (when not specified)
 
 Type: `optionalString`
 

plugin/plugins/telemetry/telemetry.go

@@ -461,7 +461,7 @@ func (p *telemetryPlugin) collectAutoTLSInfo() {
 
 func (p *telemetryPlugin) collectAutoConfInfo() {
 	p.event.AutoConf = p.config.AutoConf.Enabled.WithDefault(config.DefaultAutoConfEnabled)
-	p.event.AutoConfCustom = p.config.AutoConf.URL != config.DefaultAutoConfURL
+	p.event.AutoConfCustom = p.config.AutoConf.URL.WithDefault(config.DefaultAutoConfURL) != config.DefaultAutoConfURL
 }
 
 func (p *telemetryPlugin) collectDiscoveryInfo() {

repo/fsrepo/migrations/fs-repo-16-to-17/migration/migration.go

@@ -232,9 +232,8 @@ func enableAutoConf(confMap map[string]any) error {
 		return nil
 	}
 
-	// Add AutoConf section
+	// Add AutoConf section (URL omitted to use implicit default)
 	confMap["AutoConf"] = map[string]any{
-		"URL":                   config.DefaultAutoConfURL,
 		"Enabled":               true,
 		"Interval":              "24h",
 		"TLSInsecureSkipVerify": false,

repo/fsrepo/migrations/fs-repo-16-to-17/migration/migration_test.go

@@ -7,7 +7,6 @@ import (
 	"path/filepath"
 	"testing"
 
-	"github.com/ipfs/kubo/config"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 )
@@ -164,7 +163,9 @@ func TestMigration(t *testing.T) {
 	autoConf, exists := updatedConfig["AutoConf"]
 	assert.True(t, exists, "AutoConf section not added")
 	autoConfMap := autoConf.(map[string]interface{})
-	assert.Equal(t, config.DefaultAutoConfURL, autoConfMap["URL"], "Expected AutoConf URL")
+	// URL is not set explicitly in migration (uses implicit default)
+	_, hasURL := autoConfMap["URL"]
+	assert.False(t, hasURL, "AutoConf URL should not be explicitly set in migration")
 
 	// Check Bootstrap was updated
 	bootstrap := updatedConfig["Bootstrap"].([]interface{})

test/cli/autoconf/extensibility_test.go

@@ -179,7 +179,7 @@ func TestAutoConfExtensibility_NewSystem(t *testing.T) {
 
 	// Update config to use mock autoconf server
 	node.UpdateConfig(func(cfg *config.Config) {
-		cfg.AutoConf.URL = mockServer.URL
+		cfg.AutoConf.URL = config.NewOptionalString(mockServer.URL)
 		cfg.AutoConf.Enabled = config.True
 		cfg.AutoConf.RefreshInterval = config.NewOptionalDuration(1 * time.Second)
 		cfg.Routing.Type = config.NewOptionalString("auto") // Should enable native AminoDHT + delegated others

test/cli/migrations/migration_16_to_17_test.go

@@ -215,7 +215,6 @@ func (h *MigrationTestHelper) RequireFieldAbsent(path string) *MigrationTestHelp
 func (h *MigrationTestHelper) RequireAutoConfDefaults() *MigrationTestHelper {
 	return h.RequireFieldExists("AutoConf").
 		RequireFieldEquals("AutoConf.Enabled", true).
-		RequireFieldExists("AutoConf.URL").
 		RequireFieldEquals("AutoConf.Interval", "24h").
 		RequireFieldEquals("AutoConf.TLSInsecureSkipVerify", false)
 }