diff --git a/Dockerfile b/Dockerfile index 21aaad2..a73d32e 100644 --- a/Dockerfile +++ b/Dockerfile @@ -1,73 +1,73 @@ -# Compile the web vault using docker -# Usage: -# Quick and easy: -# `make docker-extract` -# or, if you just want to build -# `make docker` -# -# docker build -t web_vault_build . -# image_id=$(docker create web_vault_build) -# docker cp $image_id:/bw_web_vault.tar.gz . -# docker rm $image_id -# -# Note: you can use --build-arg to specify the version to build: -# docker build -t web_vault_build --build-arg VAULT_VERSION=master . - -# image_id=$(docker create bitwardenrs/web-vault@sha256:feb3f46d15738191b9043be4cdb1be2c0078ed411e7b7be73a2f4fcbca01e13c) -# docker cp $image_id:/bw_web_vault.tar.gz . -# docker rm $image_id - -FROM node:16-bullseye as build -RUN node --version && npm --version - -# Prepare the folder to enable non-root, otherwise npm will refuse to run the postinstall -RUN mkdir /vault -RUN chown node:node /vault -USER node - -# Can be a tag, release, but prefer a commit hash because it's not changeable -# https://github.com/bitwarden/clients/commit/${VAULT_VERSION} -# -# Using https://github.com/bitwarden/clients/releases/tag/web-v2023.7.1 -ARG VAULT_VERSION=42cbdbd25284460c2d9f02e3bdd8df962080b4d2 - -WORKDIR /vault -RUN git init -RUN git remote add origin https://github.com/bitwarden/clients.git -RUN git fetch --depth 1 origin "${VAULT_VERSION}" -RUN git -c advice.detachedHead=false checkout FETCH_HEAD - -COPY --chown=node:node patches /patches -COPY --chown=node:node resources /resources -COPY --chown=node:node scripts/apply_patches.sh /apply_patches.sh - -RUN bash /apply_patches.sh - -# Build -RUN npm ci -RUN npm audit fix || true - -# Switch to the web apps folder -WORKDIR /vault/apps/web - -RUN npm run dist:oss:selfhost - -RUN printf '{"version":"%s"}' \ - $(git -c 'versionsort.suffix=-' ls-remote --tags --refs --sort='v:refname' https://github.com/dani-garcia/bw_web_builds.git 'v*' | tail -n1 | grep -Eo '[^\/v]*$') \ - > build/vw-version.json - -# Delete debugging map files, optional -# RUN find build -name "*.map" -delete - -# Prepare the final archives -RUN mv build web-vault -RUN tar -czvf "bw_web_vault.tar.gz" web-vault --owner=0 --group=0 - -# We copy the final result as a separate empty image so there's no need to download all the intermediate steps -# The result is included both uncompressed and as a tar.gz, to be able to use it in the docker images and the github releases directly -FROM scratch -# hadolint ignore=DL3010 -COPY --from=build /vault/apps/web/bw_web_vault.tar.gz /bw_web_vault.tar.gz -COPY --from=build /vault/apps/web/web-vault /web-vault -# Added so docker create works, can't actually run a scratch image -CMD [""] +# Compile the web vault using docker +# Usage: +# Quick and easy: +# `make docker-extract` +# or, if you just want to build +# `make docker` +# +# docker build -t web_vault_build . +# image_id=$(docker create web_vault_build) +# docker cp $image_id:/bw_web_vault.tar.gz . +# docker rm $image_id +# +# Note: you can use --build-arg to specify the version to build: +# docker build -t web_vault_build --build-arg VAULT_VERSION=master . + +# image_id=$(docker create bitwardenrs/web-vault@sha256:feb3f46d15738191b9043be4cdb1be2c0078ed411e7b7be73a2f4fcbca01e13c) +# docker cp $image_id:/bw_web_vault.tar.gz . +# docker rm $image_id + +FROM node:16-bullseye as build +RUN node --version && npm --version + +# Prepare the folder to enable non-root, otherwise npm will refuse to run the postinstall +RUN mkdir /vault +RUN chown node:node /vault +USER node + +# Can be a tag, release, but prefer a commit hash because it's not changeable +# https://github.com/bitwarden/clients/commit/${VAULT_VERSION} +# +# Using https://github.com/bitwarden/clients/releases/tag/web-v2023.7.1 +ARG VAULT_VERSION=42cbdbd25284460c2d9f02e3bdd8df962080b4d2 + +WORKDIR /vault +RUN git init +RUN git remote add origin https://github.com/bitwarden/clients.git +RUN git fetch --depth 1 origin "${VAULT_VERSION}" +RUN git -c advice.detachedHead=false checkout FETCH_HEAD + +COPY --chown=node:node patches /patches +COPY --chown=node:node resources /resources +COPY --chown=node:node scripts/apply_patches.sh /apply_patches.sh + +RUN bash /apply_patches.sh + +# Build +RUN npm ci +RUN npm audit fix || true + +# Switch to the web apps folder +WORKDIR /vault/apps/web + +RUN npm run dist:oss:selfhost + +RUN printf '{"version":"%s"}' \ + $(git -c 'versionsort.suffix=-' ls-remote --tags --refs --sort='v:refname' https://github.com/dani-garcia/bw_web_builds.git 'v*' | tail -n1 | grep -Eo '[^\/v]*$') \ + > build/vw-version.json + +# Delete debugging map files, optional +# RUN find build -name "*.map" -delete + +# Prepare the final archives +RUN mv build web-vault +RUN tar -czvf "bw_web_vault.tar.gz" web-vault --owner=0 --group=0 + +# We copy the final result as a separate empty image so there's no need to download all the intermediate steps +# The result is included both uncompressed and as a tar.gz, to be able to use it in the docker images and the github releases directly +FROM scratch +# hadolint ignore=DL3010 +COPY --from=build /vault/apps/web/bw_web_vault.tar.gz /bw_web_vault.tar.gz +COPY --from=build /vault/apps/web/web-vault /web-vault +# Added so docker create works, can't actually run a scratch image +CMD [""] diff --git a/patches/v2023.7.1.patch b/patches/v2023.7.1.patch index 47608dd..1629716 100644 --- a/patches/v2023.7.1.patch +++ b/patches/v2023.7.1.patch @@ -53,18 +53,9 @@ index c7ac9910a..e03c8fedc 100644 diff --git a/apps/web/src/app/app.component.ts b/apps/web/src/app/app.component.ts -index 9a22ea20b..b18a9accd 100644 +index 9a22ea20b..4ce7fd4d8 100644 --- a/apps/web/src/app/app.component.ts +++ b/apps/web/src/app/app.component.ts -@@ -83,7 +83,7 @@ export class AppComponent implements OnDestroy, OnInit { - private keyConnectorService: KeyConnectorService, - private configService: ConfigServiceAbstraction, - private dialogService: DialogServiceAbstraction -- ) {} -+ ) { } - - ngOnInit() { - this.i18nService.locale$.pipe(takeUntil(this.destroy$)).subscribe((locale) => { @@ -184,6 +184,10 @@ export class AppComponent implements OnDestroy, OnInit { break; } @@ -94,11 +85,151 @@ index 29cc6df15..3dbbfbd85 100644 size: 160, }); }, 100); +diff --git a/apps/web/src/app/billing/organizations/organization-billing-history-view.component.ts b/apps/web/src/app/billing/organizations/organization-billing-history-view.component.ts +index 22cea10ac..747f534bc 100644 +--- a/apps/web/src/app/billing/organizations/organization-billing-history-view.component.ts ++++ b/apps/web/src/app/billing/organizations/organization-billing-history-view.component.ts +@@ -45,7 +45,7 @@ export class OrgBillingHistoryViewComponent implements OnInit, OnDestroy { + return; + } + this.loading = true; +- this.billing = await this.organizationApiService.getBilling(this.organizationId); ++ this.billing = null; + this.loading = false; + } + } +diff --git a/apps/web/src/app/components/environment-selector/environment-selector.component.ts b/apps/web/src/app/components/environment-selector/environment-selector.component.ts +index 9f736a72e..ee4832958 100644 +--- a/apps/web/src/app/components/environment-selector/environment-selector.component.ts ++++ b/apps/web/src/app/components/environment-selector/environment-selector.component.ts +@@ -30,7 +30,7 @@ export class EnvironmentSelectorComponent implements OnInit { + this.isEuServer = domain.includes(RegionDomain.EU); + this.isUsServer = domain.includes(RegionDomain.US) || domain.includes(RegionDomain.USQA); + this.selectedRegionImageName = this.getRegionImage(); +- this.showRegionSelector = !this.platformUtilsService.isSelfHost(); ++ this.showRegionSelector = false; + } + + getRegionImage(): string { +diff --git a/apps/web/src/app/core/init.service.ts b/apps/web/src/app/core/init.service.ts +index 4a2ef2f49..b9d6d90a7 100644 +--- a/apps/web/src/app/core/init.service.ts ++++ b/apps/web/src/app/core/init.service.ts +@@ -37,14 +37,26 @@ export class InitService { + ) {} + + init() { ++ function getBaseUrl() { ++ // If the base URL is `https://vaultwarden.example.com/base/path/`, ++ // `window.location.href` should have one of the following forms: ++ // ++ // - `https://vaultwarden.example.com/base/path/` ++ // - `https://vaultwarden.example.com/base/path/#/some/route[?queryParam=...]` ++ // ++ // We want to get to just `https://vaultwarden.example.com/base/path`. ++ let baseUrl = window.location.href; ++ baseUrl = baseUrl.replace(/#.*/, ''); // Strip off `#` and everything after. ++ baseUrl = baseUrl.replace(/\/+$/, ''); // Trim any trailing `/` chars. ++ return baseUrl; ++ } + return async () => { + // Workaround to ignore stateService.activeAccount until process.env.URLS are set + // TODO: Remove this when implementing ticket PM-2637 + this.environmentService.initialized = false; + await this.stateService.init(); + +- const urls = process.env.URLS as Urls; +- urls.base ??= this.win.location.origin; ++ const urls = { base: getBaseUrl() }; + this.environmentService.setUrls(urls); + this.environmentService.initialized = true; + +diff --git a/apps/web/src/app/core/router.service.ts b/apps/web/src/app/core/router.service.ts +index 279b94d4e..6667b7b3b 100644 +--- a/apps/web/src/app/core/router.service.ts ++++ b/apps/web/src/app/core/router.service.ts +@@ -23,7 +23,7 @@ export class RouterService { + .subscribe((event: NavigationEnd) => { + this.currentUrl = event.url; + +- let title = i18nService.t("pageTitle", "Bitwarden"); ++ let title = i18nService.t("pageTitle", "Vaultwarden"); + let child = this.activatedRoute.firstChild; + while (child.firstChild) { + child = child.firstChild; +diff --git a/apps/web/src/app/core/web-platform-utils.service.ts b/apps/web/src/app/core/web-platform-utils.service.ts +index ec829d71f..f3a362d0e 100644 +--- a/apps/web/src/app/core/web-platform-utils.service.ts ++++ b/apps/web/src/app/core/web-platform-utils.service.ts +@@ -133,15 +133,15 @@ export class WebPlatformUtilsService implements PlatformUtilsService { + } + + isDev(): boolean { +- return process.env.NODE_ENV === "development"; ++ return false; + } + + isSelfHost(): boolean { +- return WebPlatformUtilsService.isSelfHost(); ++ return false; + } + + static isSelfHost(): boolean { +- return process.env.ENV.toString() === "selfhosted"; ++ return false; + } + + copyToClipboard(text: string, options?: any): void | boolean { +diff --git a/apps/web/src/app/layouts/footer.component.html b/apps/web/src/app/layouts/footer.component.html +index 8cacb4ceb..37d7cfb03 100644 +--- a/apps/web/src/app/layouts/footer.component.html ++++ b/apps/web/src/app/layouts/footer.component.html +@@ -1,6 +1,6 @@ +