Skip to content

Commit

Permalink
fix: site settings must be accessible by non-logged-in users (#481)
Browse files Browse the repository at this point in the history
  • Loading branch information
adrians5j authored Mar 17, 2019
1 parent 8889a28 commit 780cd7b
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,6 @@ export default [
resolvers: {},
security: {
shield: {
SettingsQuery: {
googleTagManager: hasScope("cms:settings")
},
SettingsMutation: {
googleTagManager: hasScope("cms:settings")
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -102,9 +102,6 @@ export default [
},
MailchimpQuery: {
listLists: hasScope("cms:editor")
},
MailchimpMutation: {
addToList: hasScope("cms:editor")
}
}
},
Expand Down
2 changes: 1 addition & 1 deletion packages/demo-api/src/configs/development.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ export default async () => {
}
},
security: {
enabled: false,
enabled: true,
token: {
secret: process.env.WEBINY_JWT_SECRET,
expiresOn: () => addDays(new Date(), 30)
Expand Down
2 changes: 1 addition & 1 deletion packages/demo-api/src/configs/production.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ export default async (context: Object) => {
}
},
security: {
enabled: false,
enabled: true,
token: {
secret: context.jwtSecret,
expiresOn: () => addDays(new Date(), 30)
Expand Down
3 changes: 0 additions & 3 deletions packages/webiny-api-cms/src/plugins/graphql.js
Original file line number Diff line number Diff line change
Expand Up @@ -79,9 +79,6 @@ export default {
updateElement: hasScope("cms:element:crud"),
deleteElement: hasScope("cms:element:crud")
},
SettingsQuery: {
cms: hasScope("cms:settings")
},
SettingsMutation: {
cms: hasScope("cms:settings")
}
Expand Down

0 comments on commit 780cd7b

Please sign in to comment.