https://www.ory.sh/ logo
Join Slack
Powered by
Hello everyone, we are trying to implement a featu...
# ory-selfhosting
t
Hello everyone, we are trying to implement a feature in Hydra and we bumped into this Kratos issue https://github.com/ory/kratos/issues/3904. It has been open for quite some time without any feedback, is this intentional or is it a bug?
m
Hello @thankful-ocean-87883 AFAIK the current versions of Kratos and Hydra work well together. Would you be open to have a chat about how we can help you resolve this best? You can book a call with me here: meetings-eu1.hubspot.com/vincent-kraus
t
The problem is not the latest versions, it is that kratos does not allow the hydra responses to have extra params. This means that the next version of hydra will (probably) not work with this version of Kratos.
I would gladly have a chat about this, although I would prefer to kickoff the discussion on the issue
m
Ok sure, I guess the chat would be broader how we can best support you, but I will let the maintainers know to take a look at the issue. It seems it would be resolved by pointing to the v2.2.0 version.
t
oh I didn't notice a new kratos version was released.
Thank you very much, I'll have a look
@magnificent-energy-493 it's using the same lib version from that random commit
oh I realized that I misunderstood what you meant, you mean to build kratos by overriding the hydra sdk version with 
v2.2.0
? Yeah that would work. Does that mean that you are not interested in fixing this and will continue to install the hydra sdk from a random commit?
btw I tested it out and a simple replace (
go mod edit -replace <http://github.com/ory/hydra-client-go/v2=github.com/ory/hydra-client-go/v2@v2.2.0|github.com/ory/hydra-client-go/v2=github.com/ory/hydra-client-go/v2@v2.2.0>
) does not do the trick, because some fields/structs have been renamed.
@magnificent-energy-493 what would you suggest to be the way forward here? Will someone have a look at this? Should we schedule a call? Am I missing something? (I realize I could rename the fields and structs to their new names with a simple sed, but I would rather avoid such hacky solutions)
m
Hey Nikos I also replied to your DM, but wanted to followup here in case anyone else is interested. I brought it up with the product team and we will look into it as soon as possible 🙏
4 Views
Open in Slack
PreviousNext
Powered by