Differential D11701

[web] Remove authoritativeKeyserverID check from getInitialReduxState
ClosedPublic
Actions

Authored by inka on Fri, Apr 19, 8:53 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, May 1, 5:56 PM

	Unknown Object (File)
	Sat, Apr 27, 4:35 PM

	Unknown Object (File)
	Fri, Apr 26, 8:23 PM

	Unknown Object (File)
	Fri, Apr 26, 11:08 AM

	Unknown Object (File)
	Fri, Apr 26, 9:13 AM

	Unknown Object (File)
	Wed, Apr 24, 11:46 AM

	Unknown Object (File)
	Wed, Apr 24, 11:11 AM

	Unknown Object (File)
	Fri, Apr 19, 10:30 AM

View All 10 Files

Subscribers

Details

Reviewers

Commits

rCOMMfaa0659b1a25: [web] Remove authoritativeKeyserverID check from getInitialReduxState

Summary

issue: ENG-6008
This was a hack needed for testing adding keyserver back when we didn't have multikeyserver setup. As discussed in the issue, it is no longer needed

Test Plan

Have been using multikeyserver setup with this change as part of the patch diff D11141
I was able to add keyservers and communicate with them. Calling this endpoint worked correctly.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.Fri, Apr 19, 8:53 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptFri, Apr 19, 8:53 AM

Harbormaster completed remote builds in B28330: Diff 39289.Fri, Apr 19, 9:11 AM

inka requested review of this revision.Fri, Apr 19, 9:11 AM

let's goooo

This revision is now accepted and ready to land.Fri, Apr 19, 11:01 AM

Closed by commit rCOMMfaa0659b1a25: [web] Remove authoritativeKeyserverID check from getInitialReduxState (authored by inka). · Explain WhyMon, Apr 22, 4:35 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMMfaa0659b1a25: [web] Remove authoritativeKeyserverID check from getInitialReduxState.

ashoat mentioned this in D11141: [IGNORE] Client hacks for testing multiple keyservers.Mon, Apr 22, 4:46 AM

Revision Contents
Changeset List

Path

Size

web/

redux/

action-types.js

12 lines

Diff 39330

web/redux/action-types.js

Loading...