Closed
Bug 779260
Opened 13 years ago
Closed 13 years ago
Vouched Mozillians can't view API docs
Categories
(Participation Infrastructure :: Phonebook, defect)
Participation Infrastructure
Phonebook
Tracking
(Not tracked)
VERIFIED
FIXED
2012-08-08
People
(Reporter: aakashd, Assigned: aakashd)
Details
Steps to Reproduce:
1. Login as a vouched mozillian
2. Go to mozillians-dev.allizom.org/api/v1/
Actual Results:
Nothing is shown
Expected Results:
Resource schema should be shown.
Comment 1•13 years ago
|
||
Commits pushed to master at https://github.com/mozilla/mozillians
https://github.com/mozilla/mozillians/commit/c329b01bfd515d0f543c41b5acae14968899a36d
[fix bug 779260] Api docs are viewable
https://github.com/mozilla/mozillians/commit/70ce3cb506dacf68dc328f982053e573610074bf
Merge pull request #292 from sammybahamas/vouchapi
[fix bug 779260] Api docs are viewable
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Comment 2•13 years ago
|
||
Is it expected that non-vouched mozillians can view /api/v1/ as well?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
| Assignee | ||
Comment 3•13 years ago
|
||
Nope; non-vouched mozillians shouldn't be able to see /api/v1/. Only Vouched Mozillians should be able to see that page.
Comment 4•13 years ago
|
||
(In reply to Aakash Desai [:aakashd] from comment #3)
> Nope; non-vouched mozillians shouldn't be able to see /api/v1/. Only Vouched
> Mozillians should be able to see that page.
Thanks Aakash --
Reopening:
Non-vouched Mozillians are able to view the /api/v1/ json
| Assignee | ||
Comment 5•13 years ago
|
||
Actually, non-vouched users can view api/v1/. They shouldn't be able to view /api/v1/vouched/ .
Comment 6•13 years ago
|
||
bumping to resolved fixed per comment 5
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Comment 7•13 years ago
|
||
QA verified on stage --
*vouched mozillians can view /api/v1/ and /api/v1/vouched/
*non-vouched mozillians can only view /api/v1/
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•