Add information about IsStartupWizardCompleted to fix security issue #21
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
name: Pull Request
about: Create a pull request to contribute to the Helm chart collection
Description
<IsStartupWizardCompleted />
would always be set tofalse
after jellyfin started or restarted.This was due to the
system.xml
configmap not including this value, so jellyfin would default it. However, this poses not only an annoyance but a security risk (as anybody could recreate through an admin account).By adding a default value in the configmap we will have to make assumptions on the persisted state of jellyfin. Therefore, this configmap has been disabled by default and the README updated to reflect the default recommended configmap and this value and what to set it to.
Type of change
Checklist
Chart.yaml
according to semantic versioning.I have included any new or changed values in thevalues.yaml
file and documented them in the README if applicable.