Monorepo for Tangled tangled.org

spindle/config: add abilility to override motd #906

merged
opened by evan.jarrett.net targeting master from evan.jarrett.net/core: spindle-motd
0
by evan.jarrett.net 3 comments
expand 1 commit
spindle/config: add abilility to override motd

Thank you for the contribution!

I'd prefer motd file path and db path to be fixed under configurable spindle data dir (see commit), so I might eventually change the config schema later.

But this seems good to merge for now!

I'm using the config directly and specifying my own path to an embedded motd

	if _, err := motdFile.Write(motd); err != nil {
		setupLog.Error(err, "failed to write MOTD temp file")
		os.Exit(1)
	}
	motdFile.Close()
	spindleCfg.Server.MOTDFile = motdFile.Name()

If so, you can do same for {cfg.DataDir}/motd file.

// spindle/config.go
func (s Server) MotdPath() string {
	return filepath.Join(s.DataDir, "motd")
}

// ...
	if err := os.WriteFile(path, data, 0644); err != nil {
		setupLog.Error(err, "failed to write MOTD file")
		os.Exit(1)
	}

Just write embedded motd bytes to {cfg.DataDir}/motd file.

sign up or login to add to the discussion
expand 1 commit
spindle/config: add abilility to override motd

Instead of setting the files via the config, what I really want is to just set the byte content directly. I have updated the PR to reflect this.

how is the motd overridden in this scenario? SetMotd is not being called anywhere.

Its not called in tangled core. I'm using tangled as a libary and I was opening it as a way for me to set the motd directly with a []byte, rather than go through the original route I was going to do by setting the MOTD_FILE in the config. I can still go that route, but seems like its going to refactored based on boltless's commit to add datadir.

sounds good, lgtm!

pull request successfully merged
sign up or login to add to the discussion
Labels

None yet.

assignee

None yet.

Participants 3
AT URI
at://did:plc:pddp4xt5lgnv2qsegbzzs4xg/sh.tangled.repo.pull/3mahvraqqgz22