Fails to startup when a SSO connection is added.

ory / hydra

The most scalable and customizable OpenID Certified™ OpenID Connect and OAuth Provider on the market. Become an OpenID Connect and OAuth2 Provider over night. Broad support for related RFCs. Written in Go, cloud native, headless, API-first. Available as a service on Ory Network and for self-hosters.

https://www.ory.sh/?utm_source=github&utm_medium=banner&utm_campaign=hydra

Apache License 2.0

15.68k stars 1.5k forks source link

Fails to startup when a SSO connection is added. #141

Closed mfzl closed 8 years ago

mfzl commented 8 years ago

Because hydra_policies table is used for connections also. Ladon errors when it tries to decode conditions.

https://github.com/ory-am/ladon/blob/63d3e27a44b7e60f0cfd8a38231ccbe13c74aee7/manager_rethink.go#L40

aeneasr commented 8 years ago

whoops, that's a typo here: https://github.com/ory-am/hydra/blob/master/cmd/server/handler_connection_factory.go#L26

mfzl commented 8 years ago

Ah, makes sense. Seemed weird to me too, but rationalized that you might be trying to save a bit of memory by reducing number of tables as it has a small overhead.

aeneasr commented 8 years ago

no, just me not paying attention :D