Fix buildToken including our aci incorrectly

This commit is contained in:
Jamie Kyle 2024-10-10 13:18:05 -07:00 committed by GitHub
parent 48ee8e4142
commit 833723b58d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -257,8 +257,8 @@ export class GroupSendEndorsementState {
// Fast path sending to one person
if (serviceIds.size === 1) {
log.info(`${logId}: using single member endorsement`);
const [serviceId] = serviceIds;
log.info(`${logId}: using single member endorsement (${serviceId})`);
return this.#getMemberEndorsement(serviceId);
}
@ -275,15 +275,19 @@ export class GroupSendEndorsementState {
);
const otherMembers = new Set(difference);
const includesOurs = otherMembers.delete(this.#ourAci);
const includesOurs = !otherMembers.delete(this.#ourAci);
if (otherMembers.size === 0) {
log.info(`${logId}: using combined endorsement`);
log.info(
`${logId}: using combined endorsement (includesOurs: ${includesOurs})`
);
return this.#getCombinedEndorsement(includesOurs);
}
if (otherMembers.size < memberCount / 2) {
log.info(`${logId}: subtracting missing members`);
log.info(
`${logId}: subtracting missing members (includesOurs: ${includesOurs})`
);
return this.#subtractMemberEndorsements(otherMembers, includesOurs);
}