1214304 - API v2 search by group AND email results all users

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Peter Bengtsson [:peterbe]]

When I do a search like this:
https://mozillians.org/api/v2/users/?group=air+mozilla+contributors&api-key=XXXXXX&username=a-buck
or 
https://mozillians.org/api/v2/users/?group=air+mozilla+contributors&api-key=XXXXXXX&email=peterbe%40mozilla.com
I'd expect it to return the mutual exclusive. I.e. users who are part of that mentioned group AND (have that email or username). 

Instead it returns ALL users of that group ignoring my `email=...` or `username=...`

Comment 1

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/0deba8571a186aa9eb3d0cec371c17539a3fb260
[Fix bug 1214304] Fix queryset in MethodFilter filters.

https://github.com/mozilla/mozillians/commit/012680d91430cfe5e940d7555401304db7a5bbf1
Merge pull request #1257 from johngian/1214304-api-v2-results

[Fix bug 1214304] Fix queryset in MethodFilter filters.

Comment 2

[Peter Bengtsson [:peterbe]]

Is there a way I can track or be informed when this reaches prod?

Comment 3

[John Giannelos [:nemo-yiannis]]

I will update the bug. This will most probably go on prod tomorrow morning UTC.

Comment 4

[John Giannelos [:nemo-yiannis]]

Verified on stage:
Tried this query on stage:
https://mozillians.allizom.org/api/v2/users/?group=<group_name>&email=<email_address>

Returns exactly one user (with email <email_address>) if user is member of group <group_name> else no users.

Comment 5

[John Giannelos [:nemo-yiannis]]

:peterbe

This fix is live on prod.
Thank you for reporting that!

Comment 6

[Peter Bengtsson [:peterbe]]

It works! Yay!