Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated options file with singlefile options and page restructure #705

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ab623
Copy link
Contributor

@ab623 ab623 commented Apr 17, 2024

I noticed the Options doc did not contain a number of single-file options, and some option defaults were incorrect.

I have updated them as best as I can, and also added a bit more structure into the document, given the number of options.

Copy link
Owner

@sissbruecker sissbruecker left a comment

Choose a reason for hiding this comment

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

Thanks, clustering the options a bit is a good idea. Added a few comments.

@@ -25,21 +25,78 @@ All options need to be defined as environment variables in the environment that

## List of options
Copy link
Owner

Choose a reason for hiding this comment

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

Let's remove this one, seems superfluous with the new categories.


A json string with additional options for the database. Passed directly to OPTIONS.

### `LD_FAVICON_PROVIDER`
Copy link
Owner

Choose a reason for hiding this comment

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

Let's move this out of database options. I'd suggest to have a "Other options" or some other catch all category.

The password for the initial superuser.
When left undefined, the superuser will be created without a usable password, which means the user can not authenticate using credentials / through the login form, and can only be authenticated using proxy authentication (see [`LD_ENABLE_AUTH_PROXY`](#ld_enable_auth_proxy)).

## Background Tasks
Copy link
Owner

Choose a reason for hiding this comment

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

This also contains a number of things not related to background tasks. Maybe repurpose this as the "Other options" category. Would also make sense to move this to the bottom of the doc then.


### `LD_DB_USER`
### `LD_ENABLE_SNAPSHOTS`
Copy link
Owner

Choose a reason for hiding this comment

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

This should not be exposed to users, it's only an internal marker for the app to know whether the dependencies for creating snapshots are installed and can be used. Changing this in the base image would result in broken features. Let's remove this.


Alternative favicon providers:
- DuckDuckGo: `https://icons.duckduckgo.com/ip3/{domain}.ico`
## Superuser Options
Copy link
Owner

Choose a reason for hiding this comment

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

These might be the most used options, which is why I had it at the top. Let's move it back there.


### `LD_DB_PORT`
### `LD_SINGLEFILE_PATH`
Copy link
Owner

Choose a reason for hiding this comment

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

This probably shouldn't be changed either. If there's no use case for changing this it seems better to remove it to reduce confusion for readers what they should do with it.

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.

None yet

2 participants