Skip to content

Commit 64b7bac

Browse files
committed
fix(app): redirect to login only on 401s
1 parent 667fc74 commit 64b7bac

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

Diff for: templates/coffeescript/app.coffee

+2-1
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,10 @@ angular.module('<%= scriptAppName %>', [<%= angularModules %>])<% if (ngRoute) {
2222

2323
$locationProvider.html5Mode true<% if (mongoPassportUser) { %>
2424

25+
# Intercept 401s and redirect you to login
2526
$httpProvider.interceptors.push ['$q', '$location', ($q, $location) ->
2627
responseError: (response) ->
27-
if response.status is 401 or response.status is 403
28+
if response.status is 401
2829
$location.path '/login'
2930
$q.reject response
3031
else

Diff for: templates/express/controllers/users.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ exports.create = function (req, res, next) {
1111
var newUser = new User(req.body);
1212
newUser.provider = 'local';
1313
newUser.save(function(err) {
14-
if (err) return next(err);
14+
if (err) return res.json(400, err);
1515

1616
req.logIn(newUser, function(err) {
1717
if (err) return next(err);

Diff for: templates/javascript/app.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@ angular.module('<%= scriptAppName %>', [<%= angularModules %>])<% if (ngRoute) {
2626

2727
$locationProvider.html5Mode(true);<% if (mongoPassportUser) { %>
2828

29-
// Intercept 401s and 403s and redirect you to login
29+
// Intercept 401s and redirect you to login
3030
$httpProvider.interceptors.push(['$q', '$location', function($q, $location) {
3131
return {
3232
'responseError': function(response) {
33-
if(response.status === 401 || response.status === 403) {
33+
if(response.status === 401) {
3434
$location.path('/login');
3535
return $q.reject(response);
3636
}

0 commit comments

Comments
 (0)