italia / Satosa-Saml2Spid

SATOSA SAML-to-SAML proxy with Spid compliances
Apache License 2.0
42 stars 25 forks source link

OIDC mongodb index for registration_access_token #66

Closed MdreW closed 1 year ago

MdreW commented 1 year ago

in the mongodb we have an index unique for registration_access_token field, but this field is required only for the dynamic registration. I propose to add {sparse: true} in the index for skip nil value. db.client.createIndex( { "registration_access_token": 1 }, { unique: true, sparse: true } )

https://www.mongodb.com/docs/manual/core/index-sparse/

MdreW commented 1 year ago

Hi @peppelinux , if is fine I make a pull request

peppelinux commented 1 year ago

@MdreW yes it's fine but related to satosa-oidcop package and not in this project

MdreW commented 1 year ago

I @peppelinux , in satosa-oidcop you have linked the Satosa-Saml2Spid mongo db page for bd creation instruction! :smile:

At moment is needed only update our Mongo README :+1:

peppelinux commented 1 year ago

@MdreW I'm fine with a PR, if you can do that please go ahead expecting an approval :-)

MdreW commented 1 year ago

@MdreW I'm fine with a PR, if you can do that please go ahead expecting an approval :-)

I'm sorry, I was hasty :sweat_smile: