Skip to content

WIP: Enable both privval types (tcp, unix) per prefix#2847

Closed
liamsi wants to merge 1 commit intodevelopfrom
2827-privval_types
Closed

WIP: Enable both privval types (tcp, unix) per prefix#2847
liamsi wants to merge 1 commit intodevelopfrom
2827-privval_types

Conversation

@liamsi
Copy link
Contributor

@liamsi liamsi commented Nov 15, 2018

ref #2827

Some quick changes done to test the IPC / unix communication with the kms. If the general direction is OK, I would add a few tests and clean up this PR.

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

- rename privKey to secretConPrivKey to be more explicit what this is
used for
@liamsi
Copy link
Contributor Author

liamsi commented Nov 19, 2018

closing in favour of #2866. It does the same thing but has some tests and documentation.

@liamsi liamsi closed this Nov 19, 2018
@liamsi liamsi deleted the 2827-privval_types branch November 19, 2018 15:03
melekes pushed a commit that referenced this pull request Nov 21, 2018
Ref #2827

(I have since seen #2847 which is a fix for the same issue; this PR has tests and docs too ;) )
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant