fix(docs) :: remove DSN=DuckDB from the database_url default

[81reap]

create_default_database only ever builds a sqlite:// URL, so this isn't a possible default

SQLPage/src/app_config.rs

Lines 633 to 676 in 109b20d

	fn create_default_database(configuration_directory: &Path) -> String {
	let prefix = "sqlite://".to_owned();
	#[cfg(not(feature = "lambda-web"))]
	{
	let config_dir = cannonicalize_if_possible(configuration_directory);
	let old_default_db_path = PathBuf::from(DEFAULT_DATABASE_FILE);
	let default_db_path = config_dir.join(DEFAULT_DATABASE_FILE);
	if let Ok(true) = old_default_db_path.try_exists() {
	log::warn!(
	"Your sqlite database in {} is publicly accessible through your web server. Please move it to {}.",
	old_default_db_path.display(),
	default_db_path.display()
	);
	return prefix + old_default_db_path.to_str().unwrap();
	} else if let Ok(true) = default_db_path.try_exists() {
	log::debug!(
	"Using the default database file in {}",
	default_db_path.display()
	);
	return prefix + &encode_uri(&default_db_path);
	}
	// Create the default database file if we can
	if let Ok(tmp_file) = std::fs::File::create(&default_db_path) {
	log::info!(
	"No DATABASE_URL provided, {} is writable, creating a new database file.",
	default_db_path.display()
	);
	drop(tmp_file);
	if let Err(e) = std::fs::remove_file(&default_db_path) {
	log::debug!(
	"Unable to remove temporary probe file. It might have already been removed by another instance started concurrently: {e}"
	);
	}
	return prefix + &encode_uri(&default_db_path) + "?mode=rwc";
	}
	}
	log::warn!(
	"No DATABASE_URL provided, and {} is not writeable. Using a temporary in-memory SQLite database. All the data created will be lost when this server shuts down.",
	configuration_directory.display()
	);
	prefix + ":memory:?cache=shared"
	}

[lovasoa]

Maybe the docs should be clearer, but dsn=duckdb is the default in our duckdb docker images

[81reap]

it seems very unobvious thing to put in the configuration page fro all of SQLPage. furthermore, it seems silly that SQLPage is explaining that it overwrite the default db param for a docker image specifically designed for DuckDB.

from the perspective of an end user this is an exception to the rule (and not another default) that only applies to consumers of the docker image (and even then, the docker image with -duckdb suffix not the regular SQLPage image). I think that the context for overridden defaults for a single docker image should be kept in the description of that docker image. https://hub.docker.com/r/lovasoa/sqlpage

not sure where the Docker description is pulled from tho. it doesn't seem to be codified in the repo

[lovasoa]

I am open to a better wording in the documentation, but it must be concise and helpful. We can just state "context-dependent, see below" and add a short section later in the file.

[81reap]

I read through the docs again, and I moved it as a line in the ODBC Connection Strings section