Skip to content

fix: local wiki script#2576

Open
deer-wmde wants to merge 1 commit intomainfrom
de/fix-local-wiki-script
Open

fix: local wiki script#2576
deer-wmde wants to merge 1 commit intomainfrom
de/fix-local-wiki-script

Conversation

@deer-wmde
Copy link
Contributor

@deer-wmde deer-wmde commented Feb 6, 2026

fixes local convenience script

  • adjusts for https
  • uses cookies instead of token
  • includes survey data

Bug: T416720

Copy link
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've not tested this yet, however I have a suggestion around where to save the cookie file :)


# This script should create a wiki for a user account on a local wbaas cluster with the credentials below. Do not use this in production!

COOKIE_JAR=~/".wbaas.dev.cookies.txt"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A pet peeve of mine is developers cluttering up the home dir with dot files 😅

I suggest we follow the XDG Base Directory Specification and use something like $XDG_STATE_HOME.

$XDG_STATE_HOME defines the base directory relative to which user-specific state files should be stored. If $XDG_STATE_HOME is either not set or empty, a default equal to $HOME/.local/state should be used.

The $XDG_STATE_HOME contains state data that should persist between (application) restarts, but that is not important or portable enough to the user that it should be stored in $XDG_DATA_HOME.

Suggested change
COOKIE_JAR=~/".wbaas.dev.cookies.txt"
COOKIE_JAR="${XDG_STATE_HOME:-$HOME/.local/state}/wbaas.dev/cookies.txt"

Or is this something that needs to exist outside the repo? It could live inside the repo and be added to the .gitignore file?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants