Do not crash if some apis are only for admins

This commit is contained in:
Johannes Zellner
2019-05-14 16:41:08 +02:00
parent 84473dc10d
commit 2039a143ac
2 changed files with 6 additions and 0 deletions

View File

@@ -556,6 +556,8 @@ angular.module('Application').service('Client', ['$http', '$interval', 'md5', 'N
};
Client.prototype.getUpdateInfo = function (callback) {
if (!this._userInfo.admin) return callback(new Error('Not allowed'));
get('/api/v1/cloudron/update', null, function (error, data, status) {
if (error) return callback(error);
if (status !== 200) return callback(new ClientError(status, data));
@@ -2005,6 +2007,8 @@ angular.module('Application').service('Client', ['$http', '$interval', 'md5', 'N
};
Client.prototype.getSubscription = function (callback) {
if (!this._userInfo.admin) return callback(new Error('Not allowed'));
get('/api/v1/appstore/subscription', null, function (error, data, status) {
if (error) return callback(error);
if (status !== 200) return callback(new ClientError(status, data));

View File

@@ -83,6 +83,8 @@ angular.module('Application').controller('MainController', ['$scope', '$route',
// NOTE: this function is exported and called from the settings.js
$scope.updateSubscriptionStatus = function () {
if (!Client.getUserInfo().admin) return;
Client.getSubscription(function (error, subscription) {
if (error && error.statusCode === 412) return; // ignore if not yet registered
if (error) console.error(error);