Page MenuHomePhabricator
Differential D13817

[docker-compose] configure Identity endpoint for backup in commtest
ClosedPublic
Actions

Authored by kamil on Oct 30 2024, 3:28 AM.
Tags
None
Referenced Files
F6059590: D13817.id45455.diff
Sun, Apr 20, 8:32 AM
F6051866: D13817.id45663.diff
Sun, Apr 20, 6:14 AM
Unknown Object (File)
Sat, Apr 19, 2:13 AM
Unknown Object (File)
Fri, Apr 18, 12:13 PM
Unknown Object (File)
Sat, Apr 12, 3:47 PM
Unknown Object (File)
Mar 7 2025, 5:26 PM
Unknown Object (File)
Mar 7 2025, 5:25 PM
Unknown Object (File)
Mar 7 2025, 5:25 PM
View All 43 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
bartek
Commits
rCOMM0422e7595594: [docker-compose] configure Identity endpoint for backup in commtest
Summary

ENG-6145.

We need this to test querying userID based on user identitifier.

Depends on D13806

Test Plan

IDENTITY_SERVICE_ENDPOINT defined when running commtest in docker

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

kamil created this revision.Oct 30 2024, 3:28 AM
kamil held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptOct 30 2024, 3:28 AM
kamil added inline comments.Oct 30 2024, 3:29 AM
services/docker-compose.tests.yml
63 ↗(On Diff #45455)

commit hooks are making that change

kamil edited parent revisions, added: D13806: [backup-service] implement calling `FindUserID` RPC from Backup; removed: D13807: [backup-service] use `username` instead of `userID` when downloading backup.Oct 30 2024, 3:31 AM
kamil edited the summary of this revision. (Show Details)
kamil added a child revision: D13807: [backup-service] use `username` instead of `userID` when downloading backup.
Harbormaster completed remote builds in B32428: Diff 45455.Oct 30 2024, 3:47 AM
kamil published this revision for review.Oct 30 2024, 9:28 AM
varun accepted this revision.Nov 1 2024, 1:41 PM
This revision is now accepted and ready to land.Nov 1 2024, 1:41 PM
bartek accepted this revision.Nov 4 2024, 4:57 AM
Closed by commit rCOMM0422e7595594: [docker-compose] configure Identity endpoint for backup in commtest (authored by kamil). · Explain WhyNov 7 2024, 4:07 AM
This revision was automatically updated to reflect the committed changes.
kamil added a commit: rCOMM0422e7595594: [docker-compose] configure Identity endpoint for backup in commtest.

Revision Contents
Changeset List

PathSize
services/
3 lines

Diff 45663

View Options

services/docker-compose.tests.yml

Show First 20 Lines • Show All 217 Lines • ▼ Show 20 Lines return createPromiseAsJSIValue(
jsiStringToRustString(accessToken, innerRt), jsiStringToRustString(accessToken, innerRt),
currentID); currentID);
} catch (const std::exception &e) { } catch (const std::exception &e) {
error = e.what(); error = e.what();
}; };
}); });
} }
jsi::Value CommRustModule::getOutboundKeysForUserDevice( jsi::Value CommRustModule::getOutboundKeysForUser(
jsi::Runtime &rt, jsi::Runtime &rt,
jsi::String authUserID, jsi::String authUserID,
jsi::String authDeviceID, jsi::String authDeviceID,
jsi::String authAccessToken, jsi::String authAccessToken,
jsi::String userID, jsi::String userID) {
jsi::String deviceID) {
return createPromiseAsJSIValue( return createPromiseAsJSIValue(
rt, [&, this](jsi::Runtime &innerRt, std::shared_ptr<Promise> promise) { rt, [&, this](jsi::Runtime &innerRt, std::shared_ptr<Promise> promise) {
std::string error; std::string error;
try { try {
auto currentID = RustPromiseManager::instance.addPromise( auto currentID = RustPromiseManager::instance.addPromise(
promise, this->jsInvoker_, innerRt); promise, this->jsInvoker_, innerRt);
identityGetOutboundKeysForUserDevice( identityGetOutboundKeysForUser(
jsiStringToRustString(authUserID, innerRt), jsiStringToRustString(authUserID, innerRt),
jsiStringToRustString(authDeviceID, innerRt), jsiStringToRustString(authDeviceID, innerRt),
jsiStringToRustString(authAccessToken, innerRt), jsiStringToRustString(authAccessToken, innerRt),
jsiStringToRustString(userID, innerRt), jsiStringToRustString(userID, innerRt),
jsiStringToRustString(deviceID, innerRt),
currentID); currentID);
} catch (const std::exception &e) { } catch (const std::exception &e) {
error = e.what(); error = e.what();
}; };
}); });
} }
jsi::Value CommRustModule::versionSupported(jsi::Runtime &rt) { jsi::Value CommRustModule::versionSupported(jsi::Runtime &rt) {
Show All 14 Lines