Skip to content

Commit 1150d50

Browse files
authored
Merge pull request #528 from brmodeloweb/fix/view-only
Fix view only permission
2 parents 15c7d52 + 3a6c6c4 commit 1150d50

File tree

1 file changed

+7
-2
lines changed

1 file changed

+7
-2
lines changed

server_app/model/service.js

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -170,14 +170,15 @@ const findSharedModel = async (sharedId) => {
170170
return new Promise(async (resolve, reject) => {
171171
try {
172172
const model = await modelRepository.findOne({ "shareOptions._id": sharedId });
173-
if(model === null || model.shareOptions === null || !model.shareOptions.active || !model.shareOptions.importAllowed){
173+
if(model === null || model.shareOptions === null || !model.shareOptions.active){
174174
reject("unauthorized");
175175
}
176176
return resolve({
177177
"id": model.shareOptions._id,
178178
"model": model.model,
179179
"type": model.type,
180-
"name": model.name
180+
"name": model.name,
181+
"importAllowed": model.shareOptions.importAllowed,
181182
});
182183
} catch (error) {
183184
console.error(error);
@@ -203,6 +204,10 @@ const importModel = async (sharedId, userId) => {
203204
try {
204205
const sharedModel = await findSharedModel(sharedId);
205206

207+
if(!sharedModel.importAllowed){
208+
reject("unauthorized");
209+
}
210+
206211
const newModel = {
207212
"name": sharedModel.name,
208213
"type": sharedModel.type,

0 commit comments

Comments
 (0)