Page MenuHomePhabricator
Differential D4674

[keyserver] Look up uploads for a message using the container column
ClosedPublic
Actions

Authored by ashoat on Jul 28 2022, 11:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 10, 3:25 PM
Unknown Object (File)
Tue, Apr 8, 8:54 PM
Unknown Object (File)
Tue, Apr 8, 8:53 PM
Unknown Object (File)
Tue, Apr 8, 8:53 PM
Unknown Object (File)
Tue, Apr 8, 8:53 PM
Unknown Object (File)
Mar 16 2025, 10:04 AM
Unknown Object (File)
Mar 14 2025, 12:02 AM
Unknown Object (File)
Mar 13 2025, 6:48 AM
View All Files
Subscribers
• abosh
• adrian

Details

Reviewers
tomek
atul
Commits
rCOMMc26e02edac68: [keyserver] Look up uploads for a message using the container column
Summary

This is the second part of the stack to resolve ENG-1484, and is the core of it. This approach has *way* better performance.

Depends on D4673

Test Plan

I first tested this stack locally in my environment to make sure the queries worked. Then I patched it onto my production keyserver and confirmed performance was improved

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
keyserver/
src/
fetchers/
102 lines

Diff 15096

View Options

keyserver/src/fetchers/message-fetchers.js

Show All 18 Lines build:
steps: steps:
- name: Checkout - name: Checkout
uses: actions/checkout@v3 uses: actions/checkout@v3
- name: Install Developer certificate - name: Install Developer certificate
env: env:
MACOS_BUILD_CERTIFICATE_BASE64: ${{ secrets.MACOS_BUILD_CERTIFICATE_BASE64 }} MACOS_BUILD_CERTIFICATE_BASE64: ${{ secrets.MACOS_BUILD_CERTIFICATE_BASE64 }}
MACOS_BUILD_P12_PASSWORD: ${{ secrets.MACOS_BUILD_P12_PASSWORD }} MACOS_BUILD_P12_PASSWORD: ${{ secrets.MACOS_BUILD_P12_PASSWORD }}
MACOS_PROVISIONPROFILE_BASE64: ${{ secrets.MACOS_PROVISIONPROFILE_BASE64 }}
KEYCHAIN_PASSWORD: ${{ secrets.KEYCHAIN_PASSWORD }} KEYCHAIN_PASSWORD: ${{ secrets.KEYCHAIN_PASSWORD }}
run: | run: |
# create variables # create variables
CERTIFICATE_PATH=$RUNNER_TEMP/build_certificate.p12 CERTIFICATE_PATH=$RUNNER_TEMP/build_certificate.p12
KEYCHAIN_PATH=$RUNNER_TEMP/app-signing.keychain-db KEYCHAIN_PATH=$RUNNER_TEMP/app-signing.keychain-db
PROVISIONPROFILE_PATH=$GITHUB_WORKSPACE/desktop/macOS_App_Provisioning_Profile.provisionprofile
# import certificate from secrets # import certificate from secrets
echo -n "$MACOS_BUILD_CERTIFICATE_BASE64" | base64 --decode --output $CERTIFICATE_PATH echo -n "$MACOS_BUILD_CERTIFICATE_BASE64" | base64 --decode --output $CERTIFICATE_PATH
# create temporary keychain # create temporary keychain
security create-keychain -p "$KEYCHAIN_PASSWORD" $KEYCHAIN_PATH security create-keychain -p "$KEYCHAIN_PASSWORD" $KEYCHAIN_PATH
security set-keychain-settings -lut 21600 $KEYCHAIN_PATH security set-keychain-settings -lut 21600 $KEYCHAIN_PATH
security unlock-keychain -p "$KEYCHAIN_PASSWORD" $KEYCHAIN_PATH security unlock-keychain -p "$KEYCHAIN_PASSWORD" $KEYCHAIN_PATH
# import certificate to keychain # import certificate to keychain
security import $CERTIFICATE_PATH -P "$MACOS_BUILD_P12_PASSWORD" -A -t cert -f pkcs12 -k $KEYCHAIN_PATH security import $CERTIFICATE_PATH -P "$MACOS_BUILD_P12_PASSWORD" -A -t cert -f pkcs12 -k $KEYCHAIN_PATH
security list-keychain -d user -s $KEYCHAIN_PATH security list-keychain -d user -s $KEYCHAIN_PATH
# import provisioning profile from secrets
echo -n "$MACOS_PROVISIONPROFILE_BASE64" | base64 --decode --output $PROVISIONPROFILE_PATH
- name: sudo ./install_protobuf.sh - name: sudo ./install_protobuf.sh
working-directory: ./scripts working-directory: ./scripts
run: sudo ./install_protobuf.sh run: sudo ./install_protobuf.sh
- name: npm install -g yarn - name: npm install -g yarn
run: npm install -g yarn run: npm install -g yarn
- name: yarn ci-cleaninstall - name: yarn ci-cleaninstall
Show All 21 Lines