Skip to content
This repository has been archived by the owner on Dec 22, 2021. It is now read-only.

PAN-2933: Always return empty list when privacy group id doesn't exist #264

Merged
merged 2 commits into from
Jul 27, 2019

Conversation

lucassaldanha
Copy link
Member

@lucassaldanha lucassaldanha commented Jul 24, 2019

Problem
When calling eea_findPrivacyGroup for the first time before any groups have been created, the API responses with {"message":"Error finding privacy group","code":-50100}

I would have expected an empty array to be returned if no groups could be found.

Interestingly, if you create a group, delete the group and then find a group again, eea_findPrivacyGroup returns the expected response of an empty array

Copy link
Contributor

@iikirilov iikirilov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lucassaldanha lucassaldanha merged commit aa9dae7 into Consensys:master Jul 27, 2019
@lucassaldanha lucassaldanha deleted the priv-group-fix branch July 27, 2019 05:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants