Moloquent VS laravel-mongodb ( laravel passport )

[deividaspetraitis]

Hi,

Alright. I'm a little bit confused. I tried to google but with no luck. What is the difference between these two repos having in mind that both are maintained by jenssegers if I'm right?

I'm trying to use laravel passport with MongoDB. Moloquent ( https://github.com/moloquent/moloquent ): there is some provider for such purposes however in this repo there is nothing about it.

Thanks.

[jenssegers]

The moloquent repository you mentioned is not maintained by me.

[deividaspetraitis]

Ok, thanks for clarifying things up. Please feel free to close this issue since I was misled. In case anything about passport I will create new one.

[jenssegers]

Probably makes sense to add Passport support here as well :)

[deividaspetraitis]

Sure, I'm open for contribution just need to do some research. :)

[JoeyHoutenbos]

@jenssegers any update about the Passport support?

[deividaspetraitis]

@JoeyHoutenbos I believe there is no possible solution to integrate passport without modifying directly it's own files, since all models are extending elequent. I have forked passport repo and added configuration feature to be able set own models it worked like a charm and I'm using it on my projects without any problems. However my pull request to official repo was rejected. Please see my fork: https://github.com/deividaspetraitis/passport

[JoeyHoutenbos]

@deividaspetraitis , this fork has an Service Provider which fixes the problems: https://moloquent.github.io/master/#passport

[jenssegers]

@deividaspetraitis Can you link me to your PR?

[deividaspetraitis]

@JoeyHoutenbos well, they just merged passports source with moloquent source since as I mentioned you can't integrate passport with MongoDB without modifying directly it's source.
@jenssegers please see: laravel/passport#306

[JoeyHoutenbos]

@deividaspetraitis I see what you mean. Is there any reason not to solve this problem using the method provided by Moloquent?
@jenssegers Would be awesome to have Passport support as soon as possible, so maybe you can help us out?

[deividaspetraitis]

The main reason why I do not updated this issue is that changes must be made from passport side not from this package.
However in order to add default support of passport I think best way is to follow moloquent approach and include passport source to this package. However you will need to maintain two packages.

[deividaspetraitis]

@jenssegers what do you think? In case such approach would be considered I could merge my repo and create a pull request.

[JulioLago]

Any news about this?
Thanks for the hard work, loving laravel-mongodb so far, but now I want to implement passport without altering passport files :/

[azhard4int]

@deividaspetraitis @Aragami23 you can use this version of repository if you are using laravel 5.3 with v1.0 branch https://github.com/azhard4int/passport/tree/1.0/src