-
Notifications
You must be signed in to change notification settings - Fork 107
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
ERROR: yaml.scanner.ScannerError: mapping values are not allowed here (new issue) #134
Comments
Reusing the code in
But I still am getting errors
|
(Edit) Requires changes to: |
Bitwarden has changed the way they report version numbers for self-hosted installations. Fixes jakeswenson#134 Credit to @Ayitaka for the fix Tested and Verified, Updated install to 1.47.1 Signed-off-by: Donald Hoskins <grommish@gmail.com>
* BitBetter: update build.sh / update-bitwarden.sh Bitwarden has changed the way they report version numbers for self-hosted installations. Fixes #134 Credit to @Ayitaka for the fix Tested and Verified, Updated install to 1.47.1 Signed-off-by: Donald Hoskins <grommish@gmail.com> * Update build.sh Remove extraneous comment
Similar to #129
BW_VERSION="$(curl --silent https://raw.githubusercontent.com/bitwarden/self-host/master/bitwarden.sh | grep 'COREVERSION="' | sed 's/^[^"]*"//; s/".*//')"
returns an empty string.
The
https://raw.githubusercontent.com/bitwarden/self-host/master/bitwarden.sh
script,BW_VERSION
inbitwarden.sh
callscurl -sL https://go.btwrdn.co/bw-sh-versions | grep '^ *"'${1}'":' | awk -F\: '{ print $2 }' | sed -e 's/,$//' -e 's/^"//' -e 's/"$//'
which returns empty, as
bitwarden.sh
now usesCOREVERSION=$(getVersion coreVersion)
https://github.com/bitwarden/self-host/blob/master/bitwarden.sh#L49
https://go.btwrdn.co/bw-sh-versions redirects to https://raw.githubusercontent.com/bitwarden/self-host/master/version.json
which currently returns:
returns an empty string, which means no version number is populated and causes the
ScannerError
asbitwarden/bwdata/docker/docker-compose.override.yml
showsThe text was updated successfully, but these errors were encountered: