Uncaught exception in middleware

[DZ8540]

Hello there, i have a problem. I want to check user role, i do this in my service class, where i'm throwing an error. Middleware have try catch construction where i'm use this service.

and my service

But my process will stop in the console, and the server will stop. What can I do to keep my server from stopping?

[DZ8540]

And can i get role properties without model's load method?

[tomgobich]

If I had to guess I'd say it's because your this.checkAdminRole call in your middleware is missing await, since the method is async.

As for your second question, in order to get the role properties you'll either need to load it or query it separately. Typically, what I do is have a seed that explicitly sets the role's ids. Then, define an enum within my application called Roles with each role record. This way I can perform the check without needing to make an additional database call.

For example:

// contracts/Enums/Roles.ts

enum Roles {
  ADMIN = 1,
  USER = 2
}

export default Roles

// app/Services/RoleService.ts

import Roles from 'Contracts/Enums/Roles'

export default class RoleService {
  public static checkAdminRole(user: User) {
    return user.roleId === Roles.ADMIN
  }
}

[DZ8540]

Thank you very much)

[tomgobich]

Anytime!

[DZ8540]

Can be extended model or database query builder for get relationships in one method.
https://docs.adonisjs.com/guides/database/query-builder#extending-modelquerybuilder

[tomgobich]

Yeah, I don't see why not. I've never tried doing it before, but inside the macro this should be the query builder, so you can call preload inside the macro for whatever you need. In that example, they're awaiting the result of the query and returning back. So, if you want it to still be chainable after your extended method, don't await and return back this, I think.

Something like this, again keep in mind I've never attempted this so I may have something wrong here.

ModelQueryBuilder.macro('getRole', async function () {
  return this.preload('role')
})

You can also use Query Scopes to do this as well, though I don't think the usage is as clean as what you're trying to get.
https://docs.adonisjs.com/guides/models/query-scopes#document

[DZ8540]

Query scopes should be minimalistic and clean. But it's best to use a macro. I roughly did the same with the default pagination.

Example: