Skip to content

Commit 151b0b4

Browse files
authored
Merge pull request #1 from tmeasday/patch-2
Simplify authorized loader.
2 parents 884a8c5 + f621804 commit 151b0b4

File tree

1 file changed

+6
-10
lines changed

1 file changed

+6
-10
lines changed

test/output-app/README.md

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -98,16 +98,12 @@ import { queryForRoles, findByIds } from '../server/authorize';
9898
export default class <Type> {
9999
constructor(context){
100100
...
101-
this.loaders = (_user = {}, resolver = '') => ({
102-
readOne: new DataLoader(ids => new Promise( async (resolve, reject) => {
103-
try {
104-
const authQuery = queryForRoles(_user, ['admin', 'world'], ['authorId', 'coauthorsIds'], 'readOne', { User: this.context.User }, resolver);
105-
const result = await findByIds(this.collection, ids, authQuery);
106-
resolve(result);
107-
} catch (err) { reject(err); }
108-
})),
109-
});
110-
...
101+
102+
this.unauthorizedLoader = new DataLoader(ids => findByIds(this.collection, ids));
103+
104+
const { user: me, User } = context;
105+
const authQuery = queryForRoles(me, ['admin', 'world'], ['authorId', 'coauthorsIds'], 'readOne', { User }, 'findOneLoader');
106+
this.authorizedLoader = new DataLoader(ids => findByIds(this.collection, ids, authQuery));
111107
}
112108
...
113109
async getOneById(id, _user = {}, resolver = 'tweet getOneById') {

0 commit comments

Comments
 (0)