Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove SelectKeyPair and SelectedKeyPairID methods from Whisper #1748

Merged
merged 2 commits into from
Dec 30, 2019

Conversation

adambabik
Copy link
Contributor

@adambabik adambabik commented Dec 18, 2019

Remove a notion of an active key pair from Whisper. The management of the active key pair should be put on status-go rather than Whisper.

This change also removes deduplication because it is currently not used.

Closes #1747

@ghost
Copy link

ghost commented Dec 18, 2019

Pull Request Checklist

  • Have you updated the documentation, if impacted (e.g. docs.status.im)?

@adambabik adambabik requested a review from cammellos December 18, 2019 22:26
@status-im-auto
Copy link
Member

status-im-auto commented Dec 18, 2019

Jenkins Builds

Click to see older builds (15)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 73d4458 #1 2019-12-18 22:27:50 ~1 min linux 📦zip
✔️ 73d4458 #1 2019-12-18 22:32:09 ~5 min ios 📦zip
✔️ 73d4458 #1 2019-12-18 22:33:16 ~6 min android 📦aar
✔️ eb4e457 #2 2019-12-18 22:46:25 ~39 sec linux 📦zip
✔️ eb4e457 #2 2019-12-18 22:49:04 ~3 min ios 📦zip
✔️ eb4e457 #2 2019-12-18 22:51:06 ~5 min android 📦aar
✔️ 3e53d66 #3 2019-12-18 22:49:16 ~46 sec linux 📦zip
✔️ 3e53d66 #3 2019-12-18 22:51:48 ~2 min ios 📦zip
✔️ 3e53d66 #3 2019-12-18 22:56:13 ~5 min android 📦aar
✔️ 7288cd0 #4 2019-12-20 09:41:25 ~49 sec linux 📦zip
✔️ 7288cd0 #4 2019-12-20 09:43:39 ~3 min ios 📦zip
✔️ 7288cd0 #4 2019-12-20 09:45:39 ~5 min android 📦aar
✔️ ea5e224 #5 2019-12-21 00:08:17 ~49 sec linux 📦zip
✔️ ea5e224 #5 2019-12-21 00:10:06 ~2 min ios 📦zip
✔️ ea5e224 #5 2019-12-21 00:13:08 ~5 min android 📦aar
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 7e3a385 #6 2019-12-30 11:28:55 ~44 sec linux 📦zip
✔️ 7e3a385 #6 2019-12-30 11:33:18 ~5 min ios 📦zip
✔️ 7e3a385 #6 2019-12-30 11:35:44 ~7 min android 📦aar
✔️ 65d6da8 #7 2019-12-30 11:31:48 ~35 sec linux 📦zip
✔️ 65d6da8 #7 2019-12-30 11:36:53 ~3 min ios 📦zip
✔️ 65d6da8 #7 2019-12-30 11:41:30 ~5 min android 📦aar

@pedropombeiro
Copy link
Contributor

@adambabik looks like one if the HasKeyPair conditions in TestWhisperTestSuite/TestSelectedAccountOnRestart is failing in the race tests.

@pedropombeiro
Copy link
Contributor

@adambabik remember to also remove SelectedKeyPairID in nimbusWhisperWrapper.

@pedropombeiro pedropombeiro force-pushed the remove-selected-key-whisper branch from ea5e224 to 7e3a385 Compare December 30, 2019 11:27
@pedropombeiro pedropombeiro merged commit 18af917 into develop Dec 30, 2019
@delete-merged-branch delete-merged-branch bot deleted the remove-selected-key-whisper branch December 30, 2019 11:56
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove SelectKeyPair and SelectedKeyPairID from whisper
3 participants