Paths

Table of Contentst

Differential D12443

[keyserver] move mark_prekey_as_published calls inside of publishPrekeysToIdentity
ClosedPublic
Actions

Authored by varun on Fri, Jun 14, 11:29 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Jun 22, 4:41 AM

	Unknown Object (File)
	Fri, Jun 21, 11:20 AM

	Unknown Object (File)
	Thu, Jun 20, 10:24 PM

	Unknown Object (File)
	Thu, Jun 20, 7:16 PM

	Unknown Object (File)
	Thu, Jun 20, 9:40 AM

	Unknown Object (File)
	Wed, Jun 19, 1:11 AM

	Unknown Object (File)
	Tue, Jun 18, 1:19 PM

	Unknown Object (File)
	Tue, Jun 18, 1:19 PM

View All 10 Files

Subscribers

Details

Reviewers

Commits

rCOMMb7ba804bc704: [keyserver] move mark_prekey_as_published calls inside of…

Summary

we always call mark_prekey_as_published after calling publishPrekeysToIdentity (except in an old migration, which won't be run again), so we can just move the call inside the function.

Depends on D12442

Test Plan

generated new unpublished prekeys and called validateAndUploadAccountPrekeys, confirmed that the prekeys were uploaded to identity and marked as published

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

varun created this revision.Fri, Jun 14, 11:29 AM

Herald added a subscriber: tomek. · View Herald TranscriptFri, Jun 14, 11:29 AM

Harbormaster completed remote builds in B29689: Diff 41342.Fri, Jun 14, 11:46 AM

varun requested review of this revision.Fri, Jun 14, 11:46 AM

ashoat accepted this revision.Fri, Jun 14, 2:54 PM

This revision is now accepted and ready to land.Fri, Jun 14, 2:54 PM

Closed by commit rCOMMb7ba804bc704: [keyserver] move mark_prekey_as_published calls inside of… (authored by varun). · Explain WhyMon, Jun 17, 9:47 AM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMb7ba804bc704: [keyserver] move mark_prekey_as_published calls inside of….

Revision Contents
Changeset List

Path

Size

keyserver/

src/

utils/

15 lines

Diff 41413

keyserver/src/utils/olm-utils.js

Loading...