Skip to content
This repository has been archived by the owner on Feb 8, 2018. It is now read-only.

Change 403 to 401. #4197

Merged
merged 2 commits into from
Nov 21, 2016
Merged

Change 403 to 401. #4197

merged 2 commits into from
Nov 21, 2016

Conversation

mattbk
Copy link
Contributor

@mattbk mattbk commented Nov 18, 2016

Closes #4040 and probably replaces #4189. Not sure if it covers all of #4196.

@mattbk
Copy link
Contributor Author

mattbk commented Nov 18, 2016

Fa
tests/py/test_email.py::TestEmail::test_post_anon_returns_403 FAILED
tests/py/test_identity_pages.py::Tests::test_il_is_403_for_anon FAILED
tests/py/test_identity_pages.py::Tests::test_il_is_403_for_non_admin FAILED
tests/py/test_identity_pages.py::Tests::test_ip_is_403_for_anon FAILED
tests/py/test_identity_pages.py::Tests::test_ip_is_403_for_non_admin FAILED
tests/py/test_statement_json.py::Tests::test_anonymous_gets_403 FAILED
tests/py/test_username_json.py::Tests::test_anonymous_gets_403 FAILED

@mattbk mattbk added the Review label Nov 18, 2016
@mattbk mattbk mentioned this pull request Nov 18, 2016
Closed
@mattbk
Copy link
Contributor Author

mattbk commented Nov 18, 2016

Ready for review, @whit537, @nobodxbodon, et al.

@chadwhitacre chadwhitacre merged commit da80add into master Nov 21, 2016
@chadwhitacre chadwhitacre deleted the fix-403 branch November 21, 2016 14:49
@chadwhitacre
Copy link
Contributor

!m @mattbk

This was referenced Nov 21, 2016
Merged
Closed
@nobodxbodon nobodxbodon mentioned this pull request Jan 23, 2017
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattbk @chadwhitacre